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

[Feature Request] Bringing wandb logging to sb3 #480

Closed
1 task done
krishpop opened this issue Jun 16, 2021 · 3 comments
Closed
1 task done

[Feature Request] Bringing wandb logging to sb3 #480

krishpop opened this issue Jun 16, 2021 · 3 comments
Labels
duplicate This issue or pull request already exists enhancement New feature or request

Comments

@krishpop
Copy link

Important Note: We do not do technical support, nor consulting and don't answer personal questions per email.
Please post your question on the RL Discord, Reddit or Stack Overflow in that case.

🚀 Feature

Bringing wandb logging to sb3

Motivation

I have been using weights and biases for a project and added it as an output format to the loggers (in my fork) so that it could be used instead of tensorboard for logging metrics during a run (this includes uploading Video files, Images, histograms, and scalar values as well).

Pitch

I would like to add wandb as an logger output option (with an optional import given whether or not it is installed).

Alternatives

n/a

Additional context

Add any other context or screenshots about the feature request here.

### Checklist

  • I have checked that there is no similar issue in the repo (required)
@krishpop krishpop added the enhancement New feature or request label Jun 16, 2021
@Miffyli
Copy link
Collaborator

Miffyli commented Jun 16, 2021

Related issues #333 and Stable-Baselines-Team/stable-baselines3-contrib#6 , and a comment #1

No core support was included originally because of the requirement of wandb requiring online accounts and all that (but has it changed?). In any case, support through callbacks should be easy to add. In case you add this kind of support, we would welcome contributions to the contrib repo :)

@araffin araffin added the duplicate This issue or pull request already exists label Jun 16, 2021
@krishpop
Copy link
Author

Thanks for the fast response and quickly finding that related issue/comment! Indeed they still require online accounts, I can just try adding it to the contrib repo instead.

@araffin
Copy link
Member

araffin commented Aug 14, 2021

Now officially supported: https://docs.wandb.ai/guides/integrations/other/stable-baselines-3
Video: https://www.youtube.com/watch?v=ed1bqaZGOQw

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants