Skip to content

Change the default value of IntDistribution's step value - #13

Merged
c-bata merged 1 commit into
mainfrom
c-bata/sqlite3-int-step
Dec 11, 2025
Merged

c-bata merged 1 commit into
mainfrom
c-bata/sqlite3-int-step

Conversation

@c-bata

@c-bata c-bata commented Dec 11, 2025

Copy link
Copy Markdown
Member

This PR updates the type and the default value of IntDistribution's step field from Option<i64> to i64, improving the compatibility with Optuna.
https://github.com/optuna/optuna/blob/1146ac23ab667487018b6a761092afa7361a9b3d/optuna/distributions.py#L340

@c-bata
c-bata force-pushed the c-bata/sqlite3-int-step branch from 89b0a6e to 801e561 Compare December 11, 2025 05:17
@c-bata
c-bata force-pushed the c-bata/sqlite3-int-step branch from 801e561 to 3b12faf Compare December 11, 2025 07:21
@c-bata
c-bata merged commit 76b8ddb into main Dec 11, 2025
8 checks passed
@c-bata
c-bata deleted the c-bata/sqlite3-int-step branch December 11, 2025 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant