Fix seismogram shape assertion in CI smoke test

larsgeb · claude · larsgeb · commit b5e309b79957 · 2026-02-20T10:11:25.000+01:00
elastic_wave_propagate returns (n_recv, nt), not (nt, n_recv).

Co-Authored-By: Claude Sonnet 4.6 &lt;noreply@anthropic.com&gt;
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -116,7 +116,7 @@ jobs:
               ctx, vp, vs, rho, 25, 10, wvl, rx, rz,
               dx=10.0, dz=10.0, dt=1e-3, n_boundary=5
           )
-          assert sv.shape == (20, 3), f"seismogram shape wrong: {sv.shape}"
+          assert sv.shape == (3, 20), f"seismogram shape wrong: {sv.shape}"
 
           print("All smoke tests passed.")
           EOF

Original file line number	Diff line number	Diff line change
`@@ -116,7 +116,7 @@ jobs:`
`116`	`116`	`ctx, vp, vs, rho, 25, 10, wvl, rx, rz,`
`117`	`117`	`dx=10.0, dz=10.0, dt=1e-3, n_boundary=5`
`118`	`118`	`)`
`119`		`- assert sv.shape == (20, 3), f"seismogram shape wrong: {sv.shape}"`
	`119`	`+ assert sv.shape == (3, 20), f"seismogram shape wrong: {sv.shape}"`
`120`	`120`
`121`	`121`	`print("All smoke tests passed.")`
`122`	`122`	`EOF`