Skip to content

Add Optuna's sampler tests - #59

Merged
c-bata merged 7 commits into
mainfrom
feature/test-samplers-optuna
Mar 4, 2026
Merged

c-bata merged 7 commits into
mainfrom
feature/test-samplers-optuna

Conversation

@y0z

@y0z y0z commented Mar 3, 2026 •

Copy link
Copy Markdown
Member

Add Optuna's sampler tests.

  • To pass the test, log and step supports have been implemented for RandomSampler.
  • To pass the nan test, the filtering conditions in TpeSampler have been fixed.
  • The PED-ANOVA test has been fixed to ensure low > 0 when log=True.
  • TpeSampler has been fixed to allow n_startup_trials to be set without a seed.

@y0z y0z changed the title Add tests of BasicSamplerTestCase Add Optuna's sampler tests Mar 4, 2026

@c-bata c-bata left a comment

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.

LGTM!

@c-bata
c-bata merged commit 93c7a43 into main Mar 4, 2026
6 checks passed
@c-bata
c-bata deleted the feature/test-samplers-optuna branch March 4, 2026 11:49
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.

2 participants