Skip to content

Add version constraint for tensorflow-related ci#304

Merged
c-bata merged 2 commits intooptuna:mainfrom
y0z:feature/fix-ci
Mar 18, 2025
Merged

Add version constraint for tensorflow-related ci#304
c-bata merged 2 commits intooptuna:mainfrom
y0z:feature/fix-ci

Conversation

@y0z
Copy link
Copy Markdown
Member

@y0z y0z commented Mar 13, 2025

Motivation & Description of the changes

This PR temporarily fixes tensorflow, keras and tfkeras CI by adding a version constraint to tensorflow.

@y0z y0z added the CI Continuous integration. label Mar 13, 2025
@y0z
Copy link
Copy Markdown
Member Author

y0z commented Mar 13, 2025

@nabenabe0928 Could you review this PR?

Copy link
Copy Markdown
Member

@c-bata c-bata left a comment

Choose a reason for hiding this comment

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

@y0z Thank you for your pull request. I left one suggestion.

numpy<2.0.0 # TODO(nabe): Remove this constraint once this issue is resolved. https://github.com/numpy/numpy/issues/26710
optuna
tensorflow-cpu
tensorflow-cpu<2.19.0
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Could you add a todo comment to remove the upper version constraint, and open an issue to resolve this?

@c-bata c-bata assigned c-bata and unassigned nabenabe0928 Mar 18, 2025
@c-bata
Copy link
Copy Markdown
Member

c-bata commented Mar 18, 2025

Let me change the review since this PR should be prioritized to fix scheduled CI.

@y0z
Copy link
Copy Markdown
Member Author

y0z commented Mar 18, 2025

I've opened the issue #306 to track this CI issue.

Copy link
Copy Markdown
Member

@c-bata c-bata left a comment

Choose a reason for hiding this comment

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

LGTM.

@c-bata c-bata added this to the v4.3.0 milestone Mar 18, 2025
@c-bata c-bata merged commit 5bb6de8 into optuna:main Mar 18, 2025
16 checks passed
@y0z y0z deleted the feature/fix-ci branch March 18, 2025 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Continuous integration.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants