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
Replace lightgbm with PyTorch-based example to remove lightgbm dependency in visualization tutorial #5257
Replace lightgbm with PyTorch-based example to remove lightgbm dependency in visualization tutorial #5257
Conversation
…m dependency. The example of pytorch script comes from the multi-objective tutorial page: Multi-objective Optimization with Optuna
@y0z Could you review this PR? |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5257 +/- ##
==========================================
+ Coverage 89.23% 89.38% +0.15%
==========================================
Files 206 209 +3
Lines 12944 13119 +175
==========================================
+ Hits 11550 11727 +177
+ Misses 1394 1392 -2 ☔ View full report in Codecov by Sentry. |
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 contribution.
LGTM!
NIT: Although coincidentally, in the example, the first trial achieves the best accuracy. To better demonstrate the optimization history plot, another random seed involving best-value updates might be preferable.
@y0z Thank you for your review and suggestion. How about this plot that I used a different random seed? |
Thank you for your update! |
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 the PR, LGTM!
Motivation
Resolve #5255
Description of the changes
The newly added PyTorch example comes from the multi-objective tutorial page: Multi-objective Optimization with Optuna. I've removed a few hyperparameters so as not to make the figure less clear due to overlapping text.