-
Notifications
You must be signed in to change notification settings - Fork 41
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update to Optuna 3.x to get rid of deprecation message #532
Comments
It seems that Optuna has decided to continue supporting the Both of these are on the Optuna version 3 roadmap. So for now I think we should wait and see what happens on the Optuna side. |
Optuna 3 has been released (the current release is 3.1.0), the I'll keep this issue open to remind us to update to Optuna 3.x, which should get rid of the deprecation message. |
When updating now to latest Optuna release, i.e. v3.3.0, the above is not shown, but a different one is:
When just updating to Optuna 3.3.0 unit tests pass. The migration guide tells that the (new) method |
Okay, let's switch to the new method then! I don't see why it wouldn't work. |
In recent versions of Annif (which use Optuna 2.8.0), running the
annif hyperopt
command with a--jobs
parameter gives the following warning:The Optuna documentation for the 2.7.0 release explains that process-based parallelism should be used instead (tutorial). But that would require a PostgreSQL or MySQL database (SQLite is possible but not recommended).
Another option is to keep using thread-level parallelism using ThreadPoolExecutor instead of the
n_jobs
parameter. That would be easier to implement at least.The text was updated successfully, but these errors were encountered: