search

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

Fix import path #83

Closed Alnusjaponica closed 9 months ago

Alnusjaponica commented 9 months ago

Motivation

Fix CI failure in optuna/example. See https://github.com/optuna/optuna-examples/actions/runs/7845757202

Description of the changes

Fix wrong import path to try_import(), which was brought by https://github.com/optuna/optuna-integration/pull/65. with open optuna_integration._imports.try_import() as _imports: fails since optuna_integration._imports and _imports causes name collision (cf. PEP343) See https://github.com/Alnusjaponica/optuna-examples/pull/3 for the outcome.

codecov[bot] commented 9 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (33a2d2d) 64.59% compared to head (3f0344d) 64.59%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #83 +/- ## ======================================= Coverage 64.59% 64.59% ======================================= Files 29 29 Lines 2409 2409 ======================================= Hits 1556 1556 Misses 853 853 ```

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