Skip to content

CI check: 14b fix round 2 (do not merge) - #179

Merged
jeremymanning merged 1 commit into
mainfrom
ci/14b-fix2
Oct 26, 2026
Merged

jeremymanning merged 1 commit into
mainfrom
ci/14b-fix2

Conversation

@jeremymanning

Copy link
Copy Markdown
Member

CI check only (draft, will be closed). Runs the test matrix on the Task 14b fix round 2 commit: brainnet render tests measure content at the drawn (snapped) node positions instead of comparing separate renders bitwise.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Ue9wuWfgA8yRXEuEiydTVh

CI has been red since 14b:
- test_plot_network_mass_options compared pixel windows of separate renders
  bitwise, which software OpenGL on CI does not reproduce. It now checks the
  mass fractions, the drawn sizes and opacities, the darkness of the faded
  and interior nodes, and each sphere's measured diameter (with a fixed
  radius mapping so scale_by_mass shows).
- test_results_plot_network_renders / test_results_animate_network_renders
  looked for the edges at the fitted centers, but the wrappers snap nodes
  by default; on ubuntu 3.14 the fit moved them out of the 3 px window. They
  now use display_nodes, the positions actually drawn.
- test_plot_network_projects_nodes_into_mask no longer compares two renders
  bitwise; it measures both spheres' diameters at the effective centers.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Ue9wuWfgA8yRXEuEiydTVh
@jeremymanning
jeremymanning merged commit ac93de5 into main Oct 26, 2026
18 checks passed
@jeremymanning
jeremymanning deleted the ci/14b-fix2 branch October 26, 2026 01:32
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.

1 participant