-
Notifications
You must be signed in to change notification settings - Fork 2.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
Adding support for wandb #1144
Adding support for wandb #1144
Conversation
…g if a user specifies a logger other than tensorboard or wandb
Thanks for your contribution @manangoel99 Plz lint your code to pass the workflow. |
Fixed thanks! |
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.
PTAL @manangoel99 , please also use isort=4.3.21 to sort import of your code, thanks a lot.
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.
LGTM
I'm getting an error when using wandb with custom data. Error is |
hi @samithaFHSS, have you been able to resolve the above issue? If not, can you share code to reproduce this issue please? |
@scottire Sorry to bother you, this is my log. 2022-08-12 21:29:26.829 | INFO | yolox.core.trainer:before_train:136 - Model Summary: Params: 8.94M, Gflops: 26.76 File "", line 1, in
File "/root/YOLOX-docker/tools/train.py", line 118, in main File "/root/YOLOX-docker/yolox/core/trainer.py", line 74, in train File "/root/YOLOX-docker/yolox/core/trainer.py", line 183, in before_train File "/root/YOLOX-docker/yolox/utils/logger.py", line 384, in initialize_wandb_logger File "/root/YOLOX-docker/yolox/utils/logger.py", line 206, in init AttributeError: 'VOCDetection' object has no attribute 'cats' |
Hi @samithaFHSS ! The reason this error is coming up is because the logger only currently supports COCO datasets and not VOC datasets. We will try to add it ASAP |
feat(engine): support wandb logger
This PR fixes issues #1030 and #1049 . It adds support for the wandb logger.
I've added a new command line argument
--logger
which accepts "tensorboard" and "wandb" as arguments and raises an error otherwise.More wandb related arguments like
project
,entity
etc. can be provided in the CLI using the prefixwandb-
. For exampleAll the metrics are logged to the W&B dashboard along with the hyperparameters. The model checkpoints are stored as wandb artifacts along with tagging the best and latest models appropriately.
This dashboard uses yolox-tiny.