Skip to content

Change string formatting in optuna/pruners/_hyperband.py#6370

Merged
c-bata merged 3 commits intooptuna:masterfrom
eleannapapaio:refactor
Dec 9, 2025
Merged

Change string formatting in optuna/pruners/_hyperband.py#6370
c-bata merged 3 commits intooptuna:masterfrom
eleannapapaio:refactor

Conversation

@eleannapapaio
Copy link
Copy Markdown
Contributor

Motivation

I wanted to change the string formatting according to issue #6305 , and make it more readable.

Description of the changes

I replaced using "{}".format(var) with f"{var}" formatting in optuna/pruners/_hyperband.py

@eleannapapaio eleannapapaio force-pushed the refactor branch 2 times, most recently from 007007b to 3bde002 Compare December 7, 2025 12:58
@c-bata c-bata self-assigned this Dec 8, 2025
@c-bata c-bata added the code-fix Change that does not change the behavior, such as code refactoring. label Dec 8, 2025
Co-authored-by: c-bata <contact@c-bata.link>
@eleannapapaio eleannapapaio requested a review from c-bata December 8, 2025 13:35
Co-authored-by: c-bata <contact@c-bata.link>
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. Thank you for your contribution.

@c-bata c-bata merged commit a2bba80 into optuna:master Dec 9, 2025
13 checks passed
@c-bata c-bata added this to the v4.7.0 milestone Dec 9, 2025
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.

2 participants