Skip to content

Migrate LightGBM integration#5249

Merged
HideakiImamura merged 22 commits intooptuna:masterfrom
y0z:feature/migrate-lightgbm
Feb 15, 2024
Merged

Migrate LightGBM integration#5249
HideakiImamura merged 22 commits intooptuna:masterfrom
y0z:feature/migrate-lightgbm

Conversation

@y0z
Copy link
Copy Markdown
Member

@y0z y0z commented Feb 14, 2024

Motivation

Progress #4484.

Description of the changes

@y0z y0z changed the title Migrate LightGBM integration. Migrate LightGBM integration Feb 14, 2024
@codecov
Copy link
Copy Markdown

codecov bot commented Feb 14, 2024

Codecov Report

Attention: 15 lines in your changes are missing coverage. Please review.

Comparison is base (e905252) 89.83% compared to head (591a0ed) 89.31%.
Report is 5 commits behind head on master.

Files Patch % Lines
optuna/integration/lightgbm.py 0.00% 15 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5249      +/-   ##
==========================================
- Coverage   89.83%   89.31%   -0.52%     
==========================================
  Files         213      206       -7     
  Lines       13707    13020     -687     
==========================================
- Hits        12313    11629     -684     
+ Misses       1394     1391       -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@y0z y0z force-pushed the feature/migrate-lightgbm branch from dc15fe5 to 9956f66 Compare February 14, 2024 06:04
Copy link
Copy Markdown
Contributor

@nabenabe0928 nabenabe0928 left a 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, I found a critical problem we need to address, please take a close look!

@HideakiImamura
Copy link
Copy Markdown
Member

@nabenabe0928 While you have already reviewed this PR, let me assign you as a reviewer.

Copy link
Copy Markdown
Contributor

@nabenabe0928 nabenabe0928 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the modification!
I confirmed that the changes work perfectly!

$ pip install git+https://github.com/optuna/optuna-integration@main
$ pip install lightgbm
$ python -c "from optuna.integration.lightgbm import Dataset, train"

@nabenabe0928 nabenabe0928 removed their assignment Feb 15, 2024
@nabenabe0928 nabenabe0928 added the compatibility Change that breaks compatibility. label Feb 15, 2024
Copy link
Copy Markdown
Member

@HideakiImamura HideakiImamura left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR. I have several comments. PTAL.

y0z and others added 2 commits February 15, 2024 15:04
Co-authored-by: Hideaki Imamura <38826298+HideakiImamura@users.noreply.github.com>
Copy link
Copy Markdown
Member

@HideakiImamura HideakiImamura left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

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

Labels

compatibility Change that breaks compatibility.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants