Repository navigation
fix: expose Pulse profiling measurements and related hazards - #23
Merged
Merged
Conversation
Jammy2211
added
the
pending-release
…wser-completion # Conflicts: # dashboard.html
Jammy2211
removed
the
pending-release
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 exposes available runtime, breakdown, compilation and memory evidence before users choose an archive run. MGE/rectangular models without reference candidates default to recorded runtime; exact links open populated panels and retain the requested run. Axis/device/run controls preserve compatible filters and explain missing or unknown coverage.
Related model findings, shared component/method findings and uncategorized findings are visible with explicit applicability qualifications. Exact version-qualified guidance remains separate. Captured-commit links, verified same-commit shards, multi-project navigation and batch/method measurement grouping are preserved.
Scripts Changed
pulse/setup_browser.jsand.css: measurement overview, selection/recovery and qualified hazard discovery; compatible with older optional metadata and inline v2 records.tests/browser_fixture.py,tests/browser_setup.cjs,tests/test_setup_browser.py: real captured MGE/rectangular journeys, exact numeric/hash checks, metadata fallbacks, device filters and batch/method grouping regression.REFERENCE.mdand owner-generateddashboard.html.Validation
bin/pyauto-pulse board --offlineandcheck --offlinepassed against existing snapshots/receipts; no campaign, receipt, pin or scientific record changes.Coordination and readiness
Human explicitly authorized parallel work with
dashboard-freshness(PyAutoBrain#486). This PR owns setup-browser assets/tests; that task retains board/ingest/workflow source. Freshness PR #22 is now merged into this branch via 549eb5d; the HTML conflict was resolved by owner regeneration, and all Python/browser/offline checks passed on the combined sources.Heart YELLOW matches the exact previously acknowledged reason set:
manifest drift: shared-standards blocks (generated) — 2 mismatch(es) vs PyAutoMind/repos.yamlrelease validation stale: source moved since rehearsal (PyAutoNerves, PyAutoFit, PyAutoArray, PyAutoGalaxy, PyAutoLens)Standalone organ change, no upstream library dependency. Tier: judge — human
/prmmerge. This completes implementation of Phase C; live publication follows merge/deployment.Closes #21.