Add WandB support for logging.
Tensorboard is slow a buggy - sometimes I wait about a minute for it to load all the logs, sometimes it just doesn't show them.
There is no reason not to allow fairseq users to use wandb.
It is also very simple to add a new progress bar wrapper (and I already did that).
Logger extension interface (plug-in) which can allow users to use any logging method.
I already coded this feature for myself. Currently, I am using it for my experiments and it looks pretty good. You can see PR draft #1781 for more information.
I'll be happy if you can help me to understand why Windows build is failing, I did not write any platform-specific code and I have no windows computer to test it.
Anybody? =(
@Guitaricet - I can help with this and I have a Windows computer for testing =)
Awesome! You can look into this ci log. Tests are failing for windows-latest, 3.6. Everything is looking good for Ubuntu and MacOS.
Also, what kind of tests does wandb integration require (if any)? I did not find any tests for tensorboard.
@CSP197 thank you for offering help, but I just noticed that this test is failing on all PRs, including documentation improvements. So this is not a problem of this one.
@Guitaricet - Yeah, I think its a permissions issue with the Windows system.
I can run the tests successfully on my local machine...
Ran 116 tests in 166.650s
OK (skipped=17)
...but run into the same issues when running on the Github Actions workflow.
@CSP197 thank you!
Now I hope somebody from fairseq will find this feature useful.
PR is ready for review
@myleott sorry that I am asking you directly, but could you please comment if you want this feature in fairseq and accept the PR if so.
Would love to see this feature added :) Although maybe exposing a logging extension interface might be more appropriate. It would avoid core fairseq from having many cutom logging progress bars.
@BadrYoubiIdrissi I improved my implementation a bit - now it also logs the model architecture, gradient norms, and the logger output before the training. The fork is also updated. Hope you'll find it useful.
Would be really nice to see this merged!
Merged in 1a709b2a401ac8bd6d805c8a6a5f4d7f03b923ff
Most helpful comment
@CSP197 thank you!
Now I hope somebody from fairseq will find this feature useful.