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

fix: return hyper->value dict from hyperparam search #16

Merged
merged 1 commit into from
Feb 21, 2025
Merged

Conversation

andnp
Copy link
Member

@andnp andnp commented Feb 21, 2025

We previously used dictionaries to communicate the best hyperparameter configurations found by the search which is quite a bit less brittle than a tuple with magic ordering. Also a few other methods (e.g. extract_learning_curves) expect that hypers are a dict instead of a tuple, so it's best to have internal consistency here.

@andnp andnp merged commit 9e6c761 into main Feb 21, 2025
5 checks passed
@andnp andnp deleted the fix_hypers branch February 21, 2025 01:40
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.

1 participant