Skip to content

Enhance experiment configuration UI with improved metric selection and clearer optimizer handling#337

Merged
cristian-tamblay merged 1 commit into
developfrom
fix/hyperparameter-optimization
Oct 23, 2025
Merged

Enhance experiment configuration UI with improved metric selection and clearer optimizer handling#337
cristian-tamblay merged 1 commit into
developfrom
fix/hyperparameter-optimization

Conversation

@Irozuku
Copy link
Copy Markdown
Collaborator

@Irozuku Irozuku commented Oct 20, 2025

This pull request introduces several improvements to the experiment configuration UI, focusing on enforcing required fields for model optimization, enhancing the user experience with form controls, and simplifying state management for selected metrics and optimizers. The most important changes are grouped below:

Validation and Required Fields

  • Added logic to ensure that if a model has optimizers, selecting a goal metric is required before proceeding to the next step in the experiment modal. The "Next" button is now enabled only when all required fields are filled.
  • Updated the ModelsTableSelectMetric component to accept a required prop, display an error when not filled, and use the goal_metric directly from each model row for validation. [1] [2]

Conditional Rendering

  • Updated the models table to show a message ("No hyperparameter optimization") in the metric column when a model does not have optimizers, improving clarity for users.
image image

@cristian-tamblay
Copy link
Copy Markdown
Member

I hate the "Optimazer" misspell, but if it works it works

@cristian-tamblay cristian-tamblay merged commit 759b65a into develop Oct 23, 2025
18 checks passed
@cristian-tamblay cristian-tamblay deleted the fix/hyperparameter-optimization branch October 23, 2025 13:50
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.

2 participants