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

Restructuring training visualization and adding support for Weights & Biases Logger #664

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

manangoel99
Copy link

This PR redesigns the logging system in GAN without changing any of the console logging. The proposed change is to have a BaseLogger class from which new loggers can inherit. This includes the VisualDL logger, the new Weights & Biases Logger and any new custom loggers that might be added later.
Using the W&B Logger would be really easy for users. The only requirement is to set enable_wandb:True in the config.yml file. To add any other wandb related arguments like project name, entity name and so on, the users can add the wandb key.
For example:
wandb_args
This leads to all the training metrics being logged to W&B.
The dashboard:
wandb_dashboard

The models at every model saving step and evaluation step are also logged to the W&B run with the appropriate tags and metadata.

Screenshot 2022-08-04 at 3 43 02 PM

The dashboard is available here

- VisualDL and W&B Logger inherit from a BaseLogger class
- They have common functions `log_metrics`, `log_images` and more which internally call logger specific functions
- Added functionality for users to use both VisualDL and W&B simultaneously
- Added logging models to W&B as artifacts
@manangoel99
Copy link
Author

@LielinJiang @lijianshe02 I was wondering if it was possible to get some feedback

@manangoel99
Copy link
Author

@LielinJiang @LielinJiang I would love some feedback on this whenever you have the time

@manangoel99
Copy link
Author

Hi @LielinJiang @lijianshe02 It would be awesome if I could get this reviewed whenever possible

@LielinJiang
Copy link
Collaborator

Very glad to receive your pull request, we will review it as soon as possible

@jerrywgz
Copy link
Collaborator

Thx for your contribution. Could you add the function to PaddleMIX models https://github.com/PaddlePaddle/PaddleMIX/tree/develop

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants