Skip to content

Show mechanism graphs as diagrams alongside evidence tables - #71

Open
realmarcin wants to merge 1 commit into
mainfrom
fix/graph-visualizations
Open

realmarcin wants to merge 1 commit into
mainfrom
fix/graph-visualizations

Conversation

@realmarcin

Copy link
Copy Markdown
Contributor

Summary

Record mechanism sections currently show edge tables without a graph visualization. Add build-time SVG with labeled nodes, directed arrows, predicates, hover/focus details and fit/full-size controls. Keep the evidence tables below each diagram.

Fix the graph template and dynamically loaded taxon fragments. Current committed taxon records have no causal_graphs, so no published record payload changes; a representative fragment test exercises the new diagram.

The renderer uses no external graph library or browser JavaScript. It retains isolated nodes, cycles, parallel edges and explicit placeholders for unresolved endpoints. Corpus records and scientific claims are unchanged.

Validation

  • Focused SVG tests cover graph integrity, escaping, keyboard access and independent markers for multiple graphs.
  • Taxon payload, pagination, ATB and StrainInfo rendering tests pass (43 tests including SVG tests).
  • Changed Python files pass Ruff; git diff checks pass.

Part of the fleet graph visualization audit: existing TraitMech causal diagrams, CommunityMech interaction networks and the other existing corpus maps were verified in Chromium.

Copilot AI lite review requested due to automatic review settings September 16, 2026 06:17

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.

This branch has not been deployed

No deployments
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