-
-
Notifications
You must be signed in to change notification settings - Fork 616
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
Upgrading Optuna from v2.x.x to v3.0.0 #2162
Comments
Thanks @himkt! I'm labeling this issue as "blocked" until Optuna 3.0 is released. |
Any update on this request, as optuna v3 has been released? |
So sorry for that, and thank you for reminding. We will proceed that. |
Thank you |
I just asked @keisuke-umezawa to get #2360 back to active. |
Optuna is now at version 3.6.1, but optuna_sweeper dependency is still at 2.10.0. Is there any plan to update this requirement? |
I'm afraid some third-party plugins like this one currently lack proper support -- ideally someone who uses it would need to step up and help maintain it. |
I would be happy to make the changes mentioned in this issue. To start with, current latest version of Optuna is fully compatible with Optuna 2.10.x, which means Optuna dependency in this plugin can safely be updated to the 3.x without any change to the current plugin code. Would that be okay? |
Yes! We need someone to confirm it works though (I'm not sure to which extent CI covers this plugin well enough) |
I am actively using the latest version of Optuna with this plugin, and it works perfectly. It does have the deprecation warnings about some distributions which will need to be updated in the plugin, but it's not a blocker. |
I'm so sorry @odelalleau @ridvan-eksi (thank you so much @ridvan-eksi for raising your hand to help Optuna plugin dev 🙇). I'm got back online here, please let me know if I can help you. |
Sorry but it is not a feature request.
Related: optuna/optuna#2941
Summary
Thank you for integrating Optuna, hyperparameter optimization library. We, Optuna dev-team, are working on the next major version of the library Optuna v3. In Optuna v3, we will deprecate distributions used in:
hydra/plugins/hydra_optuna_sweeper/hydra_plugins/hydra_optuna_sweeper/_impl.py
Lines 33 to 42 in 61ab29a
We will introduce
FloatDistribution
andIntDistribution
. So distributions in hydra_optuna_sweeper will have to be updated:UniformDistribution(low, high)
=>FloatDistribution(low, high)
DiscreteUniformDistribution(low, high, q)
=>FloatDistribution(low, high, step)
LogUniformDistribution(low, high)
=>FloatDistribution(low, high, log=True)
IntUniformDistribution(low, high)
=>IntDistribution(low, high)
IntUniformDistribution(low, high, step)
=>IntDistribution(low, high, step)
IntLogUniformDistribution(low, high)
=>IntDistribution(low, high, log=True)
Now hydra optuna plugin installs the latest Optuna. Users will see the deprecation warnings in the Optuna v3 release. #2163 restricts the constraint to avoid such warnings. Note that it doesn't mean Optuna v3 is imcompatible with Optuna v2. It only shows warnings and deprecated distributions are internally converted to appropriate distributions.
Motivation
I open this issue to let hydra developers know about our next major release.
And also, I will open a PR to add a version constraint to avoid installing Optuna v3.
We will back here when the release is available.
If you have some concerns or questions, please feel free to contact Optuna dev-team.
🔗 optuna/optuna#2941
Additional context
Add any other context or screenshots about the feature request here.
The text was updated successfully, but these errors were encountered: