Repository navigation
Look up a padded Zalmoxis surface node at the target pressure - #1045
Merged
Merged
Conversation
On the JAX path PROTEUS rebuilds the density column from the EOS at each node's pressure and the hand-off temperature. A padded outer node (zero pressure past a surface stop) is the surface, so its density is now looked up at the target surface pressure, as Zalmoxis returns it, instead of at 0 Pa.
The Zalmoxis anchor docstring states that a zero-pressure node takes the adiabat continued to the target surface pressure once the adiabat blend reaches 1, and the linear profile at the outer radius before that.
7 of 8 tasks
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1045 +/- ##
==========================================
+ Coverage 94.44% 94.62% +0.18%
==========================================
Files 139 139
Lines 23569 23570 +1
Branches 3944 3944
==========================================
+ Hits 22259 22304 +45
+ Misses 882 863 -19
+ Partials 428 403 -25
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:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Companion of FormingWorlds/Zalmoxis#115, which gives the padded surface node of a Zalmoxis structure (P = 0 past a stop of the outward integration) its EOS density at the target surface pressure.
After a Zalmoxis solve,
zalmoxis_solverrebuilds the density and temperature on the JAX path node by node. It looked a padded node up at 0 Pa; it now looks it up attarget_surface_pressure, as Zalmoxis does. The docstring of_resolve_zalmoxis_anchor_temperaturesstates what the padded node's temperature is in each case.PROTEUS pins the Zalmoxis release with this change later, in its own PR.
Validation of changes
macOS, Python 3.12.
pytest tests/interior_struct/test_zalmoxis.pyat the head: 193 passed. With the lookup back at the node pressure, the new test fails (1 failed).Checklist