-
-
Notifications
You must be signed in to change notification settings - Fork 971
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
Add list of projects using Optuna to examples/README.md #1605
Conversation
@araffin Can you let me know if including the RL Zoo and the link is ok with you? |
Excellent, @Crissman. Thanks for including PyKEEN. |
Please do, maybe include the v3 too? |
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.
Thank you for your PR. Could you merge upstream/master
to resolve the CI failure? And also, I added a suggestion.
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.
Thank you for your update. LGTM.
Before merging this PR, I would suggest putting a note welcoming authors of packages to submit additional PRs against the readme to add a link to their repositories/projects to the list. |
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 checked that every library uses optuna.
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!
Motivation
Work with other projects that are using Optuna.
Description of the changes
Add a section to the examples/README.md which lists other projects that are using Optuna.
WIP while reaching out to the partners to ensure they're ok with being listed, and look for more.