Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix : 기존 learningRate보다 작을 때만 learningRate업데이트를 하고 있던 문제 해결 #429

Conversation

godhyzzang
Copy link
Collaborator

… 언마운트 될때마다 그 시점의 learningRate가 제대로 업데이트 됩니다.

📄 Description of the PR

🔧 What has been changed?

fix : 기존 learningRate보다 작을 때만 learningRate업데이트를 하고 있던 문제 해결하여 이제는 페이지 언마운트 될때마다 그 시점의 learningRate가 제대로 업데이트 됩니다.

📸 Screenshots / GIFs (if applicable)

제대로 LearningRate가 기록이 되고 있는 모습

image

⚠️ Precaution & Known issues

헷갈리지 말아야할점~

currentLearningRate, completedLearningRate 컬럼이 각각 존재하지만 completedLearningRate는 백엔드쪽에서 currentLearningRate보다 이전 히스토리 기록 중에서 가장 클때만 변경하기 때문에 프론트에서는 currentLearningRate만 신경쓰면 됩니다~!

image

✅ Checklist

  • UI 브랜치 같이 확인해서 이슈없는지 확인해보기
  • 함수 이름, 변수 이름만 봐도 어떤 기능을 하는지 파악할 수 있는지 (선언적인 코드인지 확인)

… 언마운트 될때마다 그 시점의 learningRate가 제대로 업데이트 됩니다.
@godhyzzang godhyzzang requested a review from smosco December 17, 2024 06:14
@godhyzzang godhyzzang self-assigned this Dec 17, 2024
@godhyzzang godhyzzang linked an issue Dec 17, 2024 that may be closed by this pull request
3 tasks
@smosco smosco merged commit 9a197f4 into develop Dec 17, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[fix] fix not recording learning progress rate on reading page
2 participants