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

Wandb config added to docs #1853

Open
wants to merge 4 commits into
base: 0.x
Choose a base branch
from
Open

Conversation

Nourollah
Copy link

Motivation

To better use mmaction2 functionality.

Modification

It's a few lines modified to add more logger hooks to docs.

modify files:
docs/tutorials/config.md
docs_zh_CN/tutorials/config.md

@CLAassistant
Copy link

CLAassistant commented Aug 29, 2022

CLA assistant check
All committers have signed the CLA.

dict(type='TextLoggerHook', by_epoch=False),
dict(type='TensorboardLoggerHook', by_epoch=False),
dict(type='WandbLoggerHook', by_epoch=False,
init_kwargs={'entity': entity, 'project': project, 'config': cfg_dict}), # The Wandb logger is also supported, It requires `wandb` to be installed.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How to define project and cfg_dict? Thanks

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's simple. project is the project name in WandB.ai, and the entity is the user entity/name in WandB.ai. I'll make a few changes to provide better docs.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also, cfg_dict is a dictionary containing parameters we need to log into WandB.

@hukkai
Copy link
Collaborator

hukkai commented Sep 1, 2022

Will go back to this in a week. Currently working on the new version release.

@ly015 ly015 requested a review from hukkai September 2, 2022 14:07
@cir7 cir7 deleted the branch open-mmlab:0.x April 10, 2023 12:26
@cir7 cir7 closed this Apr 10, 2023
@cir7 cir7 reopened this Apr 10, 2023
@cir7 cir7 changed the base branch from master to 0.x April 10, 2023 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants