Skip to content

updated _terminator_improvement.py for TYPE_CHECKING#6343

Merged
c-bata merged 4 commits intooptuna:masterfrom
satyarth7srivastava:type_check
Nov 26, 2025
Merged

updated _terminator_improvement.py for TYPE_CHECKING#6343
c-bata merged 4 commits intooptuna:masterfrom
satyarth7srivastava:type_check

Conversation

@satyarth7srivastava
Copy link
Copy Markdown
Contributor

Motivation

This PR relates to issue #6029

Description of the changes

I've updated optuna/visualization/matplotlib/_terminator_improvement.py for TYPE_CHECKING

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 the code-fix Change that does not change the behavior, such as code refactoring. label Nov 18, 2025
@c-bata
Copy link
Copy Markdown
Member

c-bata commented Nov 18, 2025

@kAIto47802 Could you review this PR?

@c-bata
Copy link
Copy Markdown
Member

c-bata commented Nov 18, 2025

@satyarth7srivastava Thank you for your pull request.

Sorry for the extra work, but due to the recent update in the branch protection rule after #6339, could you please merge the latest master branch into yours? 🙏

@satyarth7srivastava
Copy link
Copy Markdown
Contributor Author

satyarth7srivastava commented Nov 18, 2025

@c-bata Merged the latest master branch

Copy link
Copy Markdown
Collaborator

@kAIto47802 kAIto47802 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 left a comment. PTAL :octocat:

@satyarth7srivastava
Copy link
Copy Markdown
Contributor Author

@kAIto47802 I have updated the file as per your suggestion and I also had a doubt I asked in the comments there.
Hopefully this PR can be merged now.

Copy link
Copy Markdown
Collaborator

@kAIto47802 kAIto47802 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 update. LGTM
(I'm not sure why the warning wasn't emitted when you tested)

@c-bata c-bata added this to the v4.7.0 milestone Nov 26, 2025
@c-bata c-bata merged commit 39e8c60 into optuna:master Nov 26, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code-fix Change that does not change the behavior, such as code refactoring.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants