Skip to content

Add a verified OpenHCS workflow media gallery - #98

Merged
trissim merged 3 commits into
mainfrom
codex/media-gallery
Jul 31, 2026
Merged

Add a verified OpenHCS workflow media gallery#98
trissim merged 3 commits into
mainfrom
codex/media-gallery

Conversation

@trissim

@trissim trissim commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

What changed

  • Replaces the single landing-page screenshot with an accessible, responsive gallery captured from a real seven-assay OpenHCS session.
  • Shows a 12-step imported CellProfiler Comet Assay, live lazy configuration inheritance, compilation/execution progress, linked measurements and ROI features, native Napari review, native Fiji review, and a fresh local MCP state projection.
  • Ships optimized WebP stills, WebM/MP4 animations, reduced-motion posters, and a shareable GIF. The website build derives the publish set from actual HTML references so unrelated files in the asset directory cannot ship accidentally.
  • Adds typed, validation-first capture tooling for lossless X11 sources and deterministic bounded WebP/WebM/MP4/GIF derivatives, plus privacy and provenance documentation.
  • Fixes runtime boundaries exposed by the real capture: exact Official30 well selection, typed projection settings, thin skeleton ROI retention, config-derived external import validation, manager-finalized MCP polling, and Qt startup readiness.

Why

The landing page should show the real platform rather than describe it abstractly. These captures demonstrate the coherent workflow across authoring, execution, results, viewers, and agent control while preserving a reproducible evidence chain and CellProfiler example attribution.

Validation

  • Clean seven-story runtime acceptance: 7 completed, 0 failed in 271.873 seconds.
  • Real Napari and warmed Fiji streaming verified, including native ROI presentation and selected-feature linkage.
  • Independent media validation passed for all 15 derivatives, including codec, dimensions, byte budgets, duration, and source hashes.
  • Headless Chromium desktop and 390 px mobile visual review; no document overflow.
  • Browser checks verified all three WebM animations reach ready state and all three reduced-motion fallbacks display correctly.
  • 462 focused tests passed against a private clone with the superproject-pinned external packages.
  • Ruff, format, website build/reference validation, and git diff --check passed.

trissim added 3 commits July 31, 2026 02:20
Derive exact demo wells from prepared source metadata, use typed projection settings, and retain thin skeleton ROI outputs. Make orchestrator code-mode validation trust only external value types reachable from its config roots. Wait for manager finalization in MCP workflow polling and return control to Qt before reporting startup readiness.
Capture real X11 windows losslessly, derive bounded WebP, WebM, MP4, and GIF outputs from one typed manifest, and preserve hashes and probe metadata for independent validation. Document the privacy, provenance, and reproducibility workflow.
Replace the single interface still with responsive, accessible media from a verified seven-assay OpenHCS session. Include native Napari and Fiji review, local MCP state, lazy inheritance, execution, and result-review motion with reduced-motion posters and a shareable GIF. Derive published media from the HTML references so unrelated gallery files cannot ship accidentally.
Copilot AI review requested due to automatic review settings July 31, 2026 06:27

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@augmentcode

augmentcode Bot commented Jul 31, 2026

Copy link
Copy Markdown

PR Risk Analyzer Agent🛡️

👀 Human Input Needed
A pair-review briefing is ready for you.

→ Pair Review Briefing

@augmentcode augmentcode Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Deep Code Review Agent🐛

Review completed with 1 suggestions.

Fix in Cosmos

"frame_at_seconds and no fps."
)
assert self.trim is not None
if output.frame_at_seconds > self.trim.duration_seconds:

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This accepts frame_at_seconds exactly equal to trim.duration_seconds, but the trim interval is exclusive at Trim.end_seconds; that boundary value makes poster extraction seek to the first frame after the selected clip rather than a frame inside it.

Severity: medium


🤖 Was this useful? React with 👍 or 👎

@trissim
trissim merged commit 86f9f12 into main Jul 31, 2026
36 checks passed
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.

2 participants