Skip to content

[OPIK-5758] [FE] fix: persist eval suite experiment state across page refresh#6158

Merged
itamargolan merged 1 commit intomainfrom
itamarg/OPIK-5758/persist-eval-suite-playground-state
Apr 9, 2026
Merged

[OPIK-5758] [FE] fix: persist eval suite experiment state across page refresh#6158
itamargolan merged 1 commit intomainfrom
itamarg/OPIK-5758/persist-eval-suite-playground-state

Conversation

@itamargolan
Copy link
Copy Markdown
Contributor

@itamargolan itamargolan commented Apr 9, 2026

Details

Persist experimentByPromptId (prompt→experiment ID mapping) to localStorage by removing it from the Zustand partialize exclusion list. This aligns eval suite playground persistence with regular dataset runs — both now survive page refresh. Assertion results are fetched on-demand from the backend using the persisted experiment IDs.

Issues

Resolves https://comet-ml.atlassian.net/browse/OPIK-5758

Change checklist

  • Feature/bug fix implemented
  • Tested locally
  • Tested in staging

Testing

  • Open playground with an evaluation suite dataset
  • Run an experiment execution
  • Verify assertion results appear in output cells
  • Refresh the page
  • Verify assertion results are restored (not lost)
  • Verify regular dataset playground runs still persist correctly

Documentation

No documentation changes needed.

🤖 Generated with Claude Code

… refresh

Remove experimentByPromptId from the localStorage exclusion list so the
prompt-to-experiment mapping survives page refresh, aligning eval suite
persistence with regular dataset playground runs.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@itamargolan itamargolan requested a review from a team as a code owner April 9, 2026 13:43
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 9, 2026

📋 PR Linter Failed

Missing Section. The description is missing the ## Details section.


Missing Section. The description is missing the ## Change checklist section.


Missing Section. The description is missing the ## Documentation section.

@itamargolan itamargolan requested a review from miguelgrc April 9, 2026 13:46
Copy link
Copy Markdown
Collaborator

@aadereiko aadereiko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@itamargolan itamargolan merged commit 112f583 into main Apr 9, 2026
10 of 11 checks passed
@itamargolan itamargolan deleted the itamarg/OPIK-5758/persist-eval-suite-playground-state branch April 9, 2026 14:02
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