Skip to content

Conversation

@moibra05
Copy link
Contributor

@moibra05 moibra05 commented Jan 5, 2026

Because

  • Experiment errors need to be displayed on the new results page
  • Previously errors affecting a single metric would prevent the rest of the results data from be viewed even if it was error free
  • Error messages were verbose and daunting

This commit

  • Adds the error display ui

Fixes #13723

@moibra05
Copy link
Contributor Author

moibra05 commented Jan 5, 2026

image image image image

@moibra05
Copy link
Contributor Author

moibra05 commented Jan 5, 2026

At the moment the "Contact Experimenter Support" button doesn't do anything b/c im not too sure what its intended to do and i cant find any mention of it in the PRD

@yashikakhurana
Copy link
Contributor

At the moment the "Contact Experimenter Support" button doesn't do anything b/c im not too sure what its intended to do and i cant find any mention of it in the PRD

yeah that's the good point, we can discuss this what to do with it, may be slack channel #ask-experimenter 👀

Copy link
Contributor

@yashikakhurana yashikakhurana left a comment

Choose a reason for hiding this comment

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

Thank you @moibra05 looks good to me

@moibra05 moibra05 enabled auto-merge January 6, 2026 18:39
@moibra05 moibra05 added this pull request to the merge queue Jan 6, 2026
Merged via the queue into main with commit bc4325f Jan 6, 2026
17 checks passed
@moibra05 moibra05 deleted the 13723 branch January 6, 2026 19:53
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.

Surface metric errors on the new results page

3 participants