Skip to content

Conversation

@bruAristimunha
Copy link
Collaborator

Summary

  • provide a module-level OptunaSearchCVClassifier that enforces classifier tags for scikit-learn 1.7+
  • simplify search availability to a classifier-only Optuna wrapper and keep it picklable
  • add tests to verify classifier tagging and pickling for the Optuna search wrapper

Testing

  • pytest moabb/tests/test_optuna_wrapper.py
  • pre-commit run --all-files

Codex Task

@bruAristimunha bruAristimunha merged commit 5467124 into fix-pyproject-toml-typo Nov 21, 2025
0 of 2 checks passed
@bruAristimunha bruAristimunha deleted the refactor-optuna-wrapper-for-classifier-support-tr1nd8 branch November 21, 2025 08:21
bruAristimunha added a commit that referenced this pull request Nov 21, 2025
#839)

* removing poetry and modernize the installation

* updating some details about the code heath

* estimator (?)

* updating download

* updating the pyproject

* fixing docstring

* fixing bad formation

* removing circular inventory

* updating docstring

* updating

* [pre-commit.ci] auto fixes from pre-commit.com hooks

* [pre-commit.ci] auto fixes from pre-commit.com hooks

* updating

* updating the whats new file

* updating the python in the tool

* updating the utils

* [pre-commit.ci] auto fixes from pre-commit.com hooks

* updating the webpage

* trying to solve the wrapper and failing that way.

* [pre-commit.ci] auto fixes from pre-commit.com hooks

* Ensure Optuna search wrapper works with scikit-learn 1.7 (#841)

* Ensure Optuna search wrapper works with scikit-learn 1.7

* [pre-commit.ci] auto fixes from pre-commit.com hooks

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

* updating error

* fixing split

* [pre-commit.ci] auto fixes from pre-commit.com hooks

* pre-commit

* fixing test split

* more compatible with more version of matplotlib

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants