optuna / optuna-integration

Extended functionalities for Optuna in combination with third-party libraries.
https://optuna-integration.readthedocs.io/en/latest/index.html
MIT License
39 stars 30 forks source link

Migrate LightGBM integration #85

Closed y0z closed 9 months ago

y0z commented 9 months ago

Motivation

Description of the changes

DanielAvdar commented 9 months ago

@y0z please take a look at #81

codecov[bot] commented 9 months ago

Codecov Report

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

Comparison is base (33a2d2d) 64.59% compared to head (c8e48a0) 55.06%. Report is 2 commits behind head on main.

:exclamation: Current head c8e48a0 differs from pull request most recent head 07fff44. Consider uploading reports for the commit 07fff44 to get more accurate results

Files Patch % Lines
optuna_integration/_lightgbm_tuner/optimize.py 0.00% 402 Missing :warning:
optuna_integration/_lightgbm_tuner/alias.py 0.00% 32 Missing :warning:
optuna_integration/_lightgbm_tuner/_train.py 0.00% 15 Missing :warning:
optuna_integration/_lightgbm_tuner/sklearn.py 0.00% 15 Missing :warning:
optuna_integration/lightgbm.py 89.06% 7 Missing :warning:
optuna_integration/__init__.py 0.00% 3 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #85 +/- ## ========================================== - Coverage 64.59% 55.06% -9.53% ========================================== Files 29 35 +6 Lines 2409 2949 +540 ========================================== + Hits 1556 1624 +68 - Misses 853 1325 +472 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

y0z commented 9 months ago

@DanielAvdar

Thank you for your contribution. I review your PR instead of progressing this PR.

DanielAvdar commented 9 months ago

@y0z Thanks for reviewing my PR!