Repository navigation
Explore results: run picker with History, Add analysis and Export at the top - #156
Open
t0mdavid-m wants to merge 4 commits into
Open
t0mdavid-m wants to merge 4 commits into
t0mdavid-m wants to merge 4 commits into
Conversation
…plate) Same change as the template's branch claude/project-thread-4brz6t; it arrives with the next template sync as well. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FkEGemhZJqShow7AQz5c3q
The run picker sits at the top of Explore results with the buttons that act on it (Edit comparison, History, Add analysis, Export), above the view bar. History shows how the result came about, the settings that differ from the tools' defaults, the processing report, the provenance and Start again with these settings; Export results shows the same per run. The menu opens with Quickstart, the run is named on Describe experiment and shown while it runs. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FkEGemhZJqShow7AQz5c3q
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FkEGemhZJqShow7AQz5c3q
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FkEGemhZJqShow7AQz5c3q
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars.
|
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Requested by Tom · project thread
Gives OpenDDA the page structure the other OpenMS web apps use: Describe experiment → Run analysis → Explore results → Export results. On every results page, what is shown comes first, then how it is shown.
Needs OpenMS/streamlit-template#573. This branch carries the same template change as its first commit, so it runs on its own, and the next template sync makes the two identical.
What changes for users
Explore results
History (new): how the result came about, opened from Explore results and shown on each run's card on Export results.
Menu and run flow
Screenshots
Explore results, before:

Explore results, after:

History:

Add analysis:

Export results, before and after:


Run analysis, before and after:


Video, opening History and Add analysis: explore-results-top-row.mp4
The screenshots were added in one commit and removed in the next, so they are not part of the diff.
Notes for review
src/layout_manager/ui.py. Whichever PR merges second takes the template's version of that file.Tests
tests/test_viewer_history.py, which covers the timeline and the settings that differ from the defaults.Generated by Claude Code