Repository navigation
Navigate Pulse profiling results by scientific setup - #13
Merged
Merged
Conversation
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.
Summary
Pulse now opens with the shared dashboard hero, compact campaign rows and editable copy actions. Readers choose a project, dataset, model and instrument before seeing an exact recorded configuration; timing, compilation, memory, qualification and source evidence stay together. Temporal diagnostics are collapsed.
The lens registry consumes the merged v2 catalogue. Detail shards load from the captured commit, with hash and identity verification; missing links and failed loads never substitute another setup. Existing scientific qualification, campaign state, v1 readers and machine-state semantics are preserved.
Closes #12.
Scripts Changed
pulse/board.py,pulse/campaigns.py: shared theme, compact controls/campaigns and collapsed diagnostics.pulse/setup_browser.py,.js,.css: setup routing, exact-source selectors, linear metric groups and provenance-preserving detail loads.registry.yaml, receipts/snapshots and generated board/state: v2 capture from autolens_profiling main at 4832679a.tests/browser_fixture.py,tests/browser_setup.cjs, Python regressions: hermetic browser, campaign and capture checks.Test Plan
Development shipping gate
Heart RED remains
release validation FAILED (stage integrate)(score60). Current live user explicitly requested fixing PR13's Playwright-vendor link failures and merge conflicts in this Heart recovery session. Current repair authorization and bounded plan. Development repair and PR update only; previous-session merge grant is not reused. Human merge and all green CI checks remain required. No release or CI bypass.PR repair validation
Merged current main without rewriting history. Regenerated the four conflicting dashboard/receipt/snapshot files through the existing v2 producer at commit4832679a; all98 records and508 shard identities/hashes retained. Independent Sol review CLEAN.189 pytest, Ruff lint/format, offline dashboard/state check, Chromium suite and git whitespace checks passed. Executed input-selection check: all31 authored Markdown files retained;26 dependency files excluded. Exact-head CI lint37310560345 (including live lychee) and dashboard refresh37310560357 both passed; GitHub reports CLEAN. No new authored-URL exclusions added.