Skip to content

Cut ~115 seconds from the new 3D adaptivity test gates#389

Open
lmoresi wants to merge 1 commit into
developmentfrom
bugfix/adapt-test-diet
Open

Cut ~115 seconds from the new 3D adaptivity test gates#389
lmoresi wants to merge 1 commit into
developmentfrom
bugfix/adapt-test-diet

Conversation

@lmoresi

@lmoresi lmoresi commented Jul 22, 2026

Copy link
Copy Markdown
Member

A test-cost trim that was pushed to the #378 branch a few minutes after the squash-merge, so it missed the window — re-applied cleanly on current development (with the #381-#385 callback series in) and verified: the four 3D adaptivity test files pass in 45s instead of ~160s.

Two changes, no loss of coverage:

  • The similarity-class gate ran nine full uniform refinement passes in the pure-Python engine (512x cell growth, 117 seconds) to check a bound the design-stage oracle already proved at 2.87 million cells. The gate now checks the sharp part cheaply and more strictly: the class count must saturate between passes five and six (two full isotropic levels) and plateau under the theoretical ceiling of 36. Nine seconds.

  • The Canonical PETSc-sync callback fires on fancy-indexed array copies — serial warning, parallel hang/crash #376 warning-free guard and the redistribute-then-adapt composition test each ran an identical node-redistribution pass on an identical mesh; one mover run now carries both sets of assertions.

Underworld development team with AI support from Claude Code

… mover run

The similarity-class gate ran nine full uniform passes in the pure-
Python engine (512x cell growth, 117s) to check a bound the stage-1a
oracle already proved at 2.87M cells. The CI gate now checks the sharp
part cheaply: the class count must SATURATE between passes five and six
(two full isotropic levels) and plateau under the theoretical 36 — a
stronger assertion than the ceiling alone, at 9s.

The #376 warning-free guard and the redistribute-then-adapt composition
test ran an identical mover on an identical mesh; one run now carries
both sets of assertions.

New 3D test files: ~162s -> ~48s. All 20 pass.

Underworld development team with AI support from Claude Code
Copilot AI review requested due to automatic review settings July 22, 2026 08:44

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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.

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