Skip to content

Results analysis app: small improvements#52

Merged
fanny-riols merged 17 commits intomainfrom
pr/fr/analysis-app-multi-output
Apr 15, 2026
Merged

Results analysis app: small improvements#52
fanny-riols merged 17 commits intomainfrom
pr/fr/analysis-app-multi-output

Conversation

@fanny-riols
Copy link
Copy Markdown
Collaborator

@fanny-riols fanny-riols commented Apr 10, 2026

  • Enable multiple output folders
  • Toggle to select only the latest run for selected systems
  • In results table, add a column for system name and one for timestamp
  • Add toggle to hide failed attempts in run overview per-record table
  • Split cross-run comparison table into Accuracy, EVA-X, and Diagnostic sections
  • UI improvements in Record Detail

@fanny-riols fanny-riols force-pushed the pr/fr/analysis-app-multi-output branch from ea00a75 to 234da05 Compare April 10, 2026 13:47
@fanny-riols fanny-riols force-pushed the pr/fr/analysis-app-multi-output branch from 234da05 to 4b9c055 Compare April 10, 2026 13:56
fanny-riols and others added 13 commits April 10, 2026 10:54
Failed attempt rows (from *_failed_attempt_* directories) are now flagged
in _collect_run_metrics and filtered out by default. A toggle appears above
the table when failed attempts exist, allowing the user to show them.
… sections

Adds an Output Dir column showing the parent directory of each run, and
splits the single metrics table into three labelled subtables: Accuracy
(EVA-A composites + [Accuracy] metrics), EVA-X (EVA-X composites +
[Experience] metrics), and Diagnostic & Other (remaining metrics).
Each subtable is skipped if it has no data for the current run set.
@fanny-riols fanny-riols marked this pull request as ready for review April 14, 2026 16:34
Comment thread apps/analysis.py
Comment thread apps/README.md Outdated
Copy link
Copy Markdown
Collaborator

@gabegma gabegma left a comment

Choose a reason for hiding this comment

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

Super!! Thank you, love the changes!

- Revert uv run from README (not needed with a set Python interpreter)
- Hide Output Dir column in cross-run table when only one output dir is in use
@fanny-riols fanny-riols enabled auto-merge April 15, 2026 13:06
@fanny-riols fanny-riols added this pull request to the merge queue Apr 15, 2026
Merged via the queue into main with commit 2afbc8f Apr 15, 2026
1 check passed
@fanny-riols fanny-riols deleted the pr/fr/analysis-app-multi-output branch April 15, 2026 13:11
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.

3 participants