Skip to content

Avoid latest PyTorch lightning until integration is updated#3417

Merged
HideakiImamura merged 1 commit intooptuna:masterfrom
hvy:dependency-pin-pytorch-lightning-160
Mar 30, 2022
Merged

Avoid latest PyTorch lightning until integration is updated#3417
HideakiImamura merged 1 commit intooptuna:masterfrom
hvy:dependency-pin-pytorch-lightning-160

Conversation

@hvy
Copy link
Copy Markdown
Member

@hvy hvy commented Mar 30, 2022

Motivation

Temporarily avoid the latest pytorch-lightning 1.6.0 to circumvent CI failures in the master branch.
A proper fix involves updating PyTorchLightningPruningCallback, preferably not to use private APIs.

Failure: https://github.com/optuna/optuna/runs/5745775785?check_suite_focus=true

Release note: https://github.com/PyTorchLightning/pytorch-lightning/releases/tag/1.6.0

Description of the changes

See above.

@hvy hvy added the installation Installation and dependency. label Mar 30, 2022
@himkt himkt self-assigned this Mar 30, 2022
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #3417 (f1d327e) into master (d529ffe) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3417   +/-   ##
=======================================
  Coverage   91.80%   91.80%           
=======================================
  Files         156      156           
  Lines       12262    12262           
=======================================
  Hits        11257    11257           
  Misses       1005     1005           

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@hvy
Copy link
Copy Markdown
Member Author

hvy commented Mar 30, 2022

@himkt thanks for the self assignment, let me know if you have any concerns with this change.
@HideakiImamura could you jump in as a reviewer too, although a single reviewer might suffice given #3413.

If this PR is merged, let me create a tracking issue to resolve this underlying problem. Edit: @himkt kindly created #3418

Copy link
Copy Markdown
Member

@himkt himkt 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 patch, LGTM.

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.

@HideakiImamura HideakiImamura merged commit f70f7a3 into optuna:master Mar 30, 2022
@HideakiImamura HideakiImamura added this to the v3.0.0-b0 milestone Mar 30, 2022
@hvy hvy deleted the dependency-pin-pytorch-lightning-160 branch March 30, 2022 02:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

installation Installation and dependency.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants