Skip to content

fix: plant orbit-hist fault under project root for v0.43.0 - #39

Merged
InauguralPhysicist merged 2 commits into
mainfrom
fix/orbit-hist-plant-under-project-root
Sep 9, 2026
Merged

fix: plant orbit-hist fault under project root for v0.43.0#39
InauguralPhysicist merged 2 commits into
mainfrom
fix/orbit-hist-plant-under-project-root

Conversation

@InauguralPhysicist

Copy link
Copy Markdown
Contributor

Summary

  • Dynamics CI red since EIGS_REF→v0.43.0 (run 34064151409): suite dies after the first test_orbit_hist.sh PASS with no FAIL line.
  • Root cause: planted-fault copy ran from $TMP/.../nocap.eigs. EigenScript #1106 no longer resolves import orbit via cwd; without an eigs.json walk from /tmp, the fault run aborts and set -e fails the job.
  • Fix: write the plant under tests/ so project-root resolution still finds orbit.

Test plan

  • CI suite green on this PR at EIGS_REF=v0.43.0
  • Log shows both orbit-hist PASSes (including planted fault caught)

EigenScript #1106 dropped cwd load_file/import resolution. The planted
fault wrote tests/orbit_hist_dump.eigs into $TMP and ran it from there;
`import orbit` then failed (no eigs.json walk from /tmp), and set -e
exited the suite right after the first PASS with no FAIL line.

Keep the plant under tests/ so project-root resolution still finds
orbit. Explains dynamics CI red on the v0.43.0 pin bump
(run 34064151409) and the matching Dependabot CI red.
Same #1106 class as orbit-hist: bifurcation (and bif_mem) plant() copies
under $TMP without eigs.json, so `import logistic` / module loads fail
before assertions run — suite red after the orbit-hist fix (dynamics#39).

Copy eigs.json into every planted TMP tree (bifurcation, bif_mem, lint).
@InauguralPhysicist

Copy link
Copy Markdown
Contributor Author

Follow-up: planted-fault TMP trees now get eigs.json (bifurcation/bif_mem/lint) — same #1106 class as orbit-hist.

@InauguralPhysicist
InauguralPhysicist merged commit 25603db into main Sep 9, 2026
1 check passed
@InauguralPhysicist
InauguralPhysicist deleted the fix/orbit-hist-plant-under-project-root branch September 9, 2026 07:16
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