-
Notifications
You must be signed in to change notification settings - Fork 1.6k
[OPIK-7042] [FE] feat: Optimization Studio new-run sidebar #7291
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
Merged
Merged
Changes from all commits
Commits
Show all changes
21 commits
Select commit
Hold shift + click to select a range
527bea5
[OPIK-7042] [FE] feat: Optimization new-run sidebar + metric/schema c…
awkoy 49bf5ce
feat(optimizations): new-run form as a sidebar over the runs list
awkoy 4eb15c1
refactor(optimizations): on-demand dataset name lookup + clean schema…
awkoy 6750d0a
refactor(optimizations): split new-run god-hook into per-section hooks
awkoy 5190709
[OPIK-7042] [FE] feat: new-run sidebar — non-blocking dataset prep, i…
awkoy 2865dff
[OPIK-7042] [FE] fix: address Baz review on new-run sidebar
awkoy fc32aad
[OPIK-7042] [FE] fix: address Baz re-review (round 2) on new-run sidebar
awkoy b8e85d7
[OPIK-7042] [FE] refactor: move optimization icon accents to theme to…
awkoy ce4a5fd
[OPIK-7042] [FE] fix: validate new-run form on submit + align referen…
awkoy eb84c94
[OPIK-7042] [FE] fix(new-run): drive submit + lazy validation through…
awkoy c6fa3cb
[OPIK-7042] [FE] fix(new-run): swallow already-toasted submit rejection
awkoy 6cf06cf
[OPIK-7042] [FE] refactor(new-run): use idiomatic <form onSubmit> submit
awkoy b989c49
[OPIK-7042] [FE] chore(optimizations): drop untested custom-code metr…
awkoy 7298d7e
[OPIK-7042] [FE] refactor(new-run): prune over-explanatory comments a…
awkoy e581b36
[OPIK-7042] [FE] refactor(new-run): move optimizer/metric icon maps t…
awkoy 49d6a55
[OPIK-7042] [FE] fix(new-run): stop form buttons from submitting; met…
awkoy 5c12cee
[OPIK-7042] [FE] fix(new-run): algorithm-model inheritance, metric de…
awkoy ec870fe
[OPIK-7042] [FE] revert(new-run): drop SelectItem size prop (to be do…
awkoy dacd0a7
Merge origin/main into awkoy/opik-7042-new-run-sidebar-local
awkoy da2b23c
Merge remote-tracking branch 'origin/main' into awkoy/opik-7042-new-r…
awkoy f062644
[OPIK-7042] [FE] fix(optimizations): per-field message errors + Algor…
awkoy File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Untested prompt parsing branches
src/lib/prompt.test.tsdoesn’t coversafelyGetPromptVariables()or the newmissingDatasetVariablesbranch, so regressions in dotted tags, empty input, or malformed templates could slip through — should we add unit coverage here, as.agents/skills/opik-frontend/testing.mdsuggests?Want Baz to fix this for you? Activate Fixer You can also update your AI coding guidelines based on this comment by
apply pr to [branch name]Other fix methods
Prompt for AI Agents