fix: preserve ScalarDiffusivity halo size during conversion - #6093
Conversation
|
Oh, I thought we already had solved this. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #6093 +/- ##
==========================================
- Coverage 75.20% 74.20% -1.01%
==========================================
Files 433 435 +2
Lines 27195 27692 +497
==========================================
+ Hits 20453 20549 +96
- Misses 6742 7143 +401
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
Merged current upstream The old CPU failure came from the inherited predictor-mask path. This update includes upstream's #6101 revert and later #6109 allocation changes. The unchanged allocation selector now passes locally, including the immersed nonhydrostatic case at 128 bytes against a 141-byte limit. The allocation changes come from upstream. Closures passed 343/343, allocations 47/47, and Aqua 107/107, each with exit zero. Local validation used Julia 1.12.7 on macOS arm64; the failed hosted job used Julia 1.13 on Linux. The new head still needs hosted validation. Codex used. |
Fixes #5499.
Adapting a
ScalarDiffusivityclosure substitutes the diffusivity type for its required halo size. This can make model construction fail when it compares the halo requirement with an integer.Read the halo size from the third type parameter in both
Adapt.adapt_structureandon_architecture. The regression tests check both conversion paths, coefficient preservation, and the formulation and time discretization.Validation on
dae38053:turbulence_closures/closurespassed 343 assertions,memory_allocationpassed 47, andunit/quality_assurancepassed 107. All three processes exited successfully. These CPU runs used Julia 1.12.7 on macOS arm64. Hosted Linux and GPU results remain separate.The original baseline checks reproduced both numeric conversion failures. Repaired-source checks covered numeric and callable diffusivities and model halo setup. Those baseline observations belong to the original submission.
Codex used.