Skip to content
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

{data, tools}[foss/2023a] Hydra v1.3.2, optuna v2.10.1 #19244

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

ItIsI-Orient
Copy link
Collaborator

(created using eb --new-pr)

@smoors
Copy link
Contributor

smoors commented Nov 17, 2023

@ItIsI-Orient the list of extensions is quite long, please split Optuna into its own easyconfig?

@ItIsI-Orient ItIsI-Orient changed the title {data}[foss/2023a] Hydra v1.3.2 {data, tools}[foss/2023a] Hydra v1.3.2, optuna v2.10.1 Nov 22, 2023
@easybuilders easybuilders deleted a comment from boegelbot Nov 27, 2023
@easybuilders easybuilders deleted a comment from boegelbot Nov 27, 2023
@smoors
Copy link
Contributor

smoors commented Nov 27, 2023

@ItIsI-Orient is it really needed to use the old 2.10.1 version of optuna here?
optuna is already at version 3.4.0, there is already Optuna-3.1.0-foss-2022a.eb in the EB repo.
the latter easyconfig also contains a lot less extensions.

@ItIsI-Orient
Copy link
Collaborator Author

@smoors hydra-optuna-sweeper has a requirement of optuna<3.0.0 and 2.10.1 is the most recent stable release, that meets the requirement. I haven't tried to bypass the requirements because I am not sure about the functionality/stability of the final build. Should I still try to bypass them?

@ItIsI-Orient
Copy link
Collaborator Author

From what I could find, there were some changes in Optuna v3.x.x that are yet not compatible with Hydra (see facebookresearch/hydra#2162). There is apparently a dev version of hydra-optuna-sweeper that is compatible with higher versions of Optuna (facebookresearch/hydra#2360 (comment)) but I think that we should wait until a stable release is ready.

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