-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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 validation loss logging #1494
Fix validation loss logging #1494
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1494 +/- ##
=======================================
Coverage 90.30% 90.31%
=======================================
Files 140 140
Lines 8335 8339 +4
Branches 1400 1401 +1
=======================================
+ Hits 7527 7531 +4
Misses 570 570
Partials 238 238
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
* Add parameter safe_serialization to DiffusionPipeline.save_pretrained * Add option safe_serialization on ModelMixin.save_pretrained * Add test test_save_safe_serialization * Black * Re-trigger the CI * Fix doc-builder * Validate files are saved as safetensor in test_save_safe_serialization
mfernezir,您好!您在MMSeg项目中给我们提的PR非常重要,感谢您付出私人时间帮助改进开源项目,相信很多开发者会从你的PR中受益。 |
Motivation
The validation loss is currently not logged in the setting of both train and val workflow modes.
The problem is described in this issue:
#1396
The training log in master
The training log in this pr
Modification
val_step
is modified to add_val
suffix to all validation losses, to distinguish those losses from those in the training step. This enables both training and validation loss logging.