-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[Docs] Update Config File to Added WandBLogger Hook Details #1606
base: master
Are you sure you want to change the base?
Conversation
interval=50, # Interval to print the log | ||
hooks=[ | ||
dict(type='TextLoggerHook', by_epoch=False), # The logger used to record the training process | ||
dict(type='TensorboardLoggerHook', by_epoch=False), # The Tensorboard logger to record log offline |
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.
Why by_epoch
is set to False by default?
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.
I used this parameter to show the possibility of changing it to True or False to those who want to use it. It is possible to give it the True default value, I don't have any special reason to use the False value as default.
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.
I think it's better not to change the default setting here. To show the options, we can add by_epoch=True
here.
|
||
``` |
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.
The indent seems incorrect here.
Motivation
To better use MMPose functionality.
Modification
It's a few lines modified to add more logger hooks to docs.