-
Notifications
You must be signed in to change notification settings - Fork 462
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] PARSeq tensorflow fixes #1228
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, just one point to clarify
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks, LGTM
c092be3
to
c19eb9b
Compare
Codecov Report
@@ Coverage Diff @@
## main #1228 +/- ##
==========================================
- Coverage 93.55% 93.32% -0.23%
==========================================
Files 154 154
Lines 6915 6924 +9
==========================================
- Hits 6469 6462 -7
- Misses 446 462 +16
Flags with carried forward coverage won't be shown. Click here to find out more.
|
This PR:
several fixes
test run
torch fail will be fixed if #1227 is merged
Validation set loaded in 0.002758s (2520 samples in 79 batches)
Train set loaded in 0.003837s (126000 samples in 3937 batches)
Validation loss decreased inf --> 4.64792: saving state...
Epoch 1/20 - Validation loss: 4.64792 (Exact: 0.08% | Partial: 0.08%)
Validation loss decreased 4.64792 --> 4.37974: saving state...
Epoch 2/20 - Validation loss: 4.37974 (Exact: 0.48% | Partial: 0.91%)
Validation loss decreased 4.37974 --> 4.3734: saving state...
Epoch 3/20 - Validation loss: 4.3734 (Exact: 0.99% | Partial: 1.39%)
Validation loss decreased 4.3734 --> 4.32947: saving state...
Epoch 4/20 - Validation loss: 4.32947 (Exact: 2.14% | Partial: 3.10%)