-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for T_from_p_h in a few fluid properties #29621
base: next
Are you sure you want to change the base?
Conversation
Use a single Newton inversion rather than 3 calls through SFP definition of T(p,h) refs idaholab#29620
as a Newton inversion of h(p,T) in HelmhotzFP refs idaholab#29620
a derived class of HelmhotzHP refs idaholab#29620
Job Documentation, step Docs: sync website on f7a3ba3 wanted to post the following: View the site here This comment will be updated on new commits. |
test failures are fake news |
Beautiful expressions. They pass the consistency check at least. So I'll approve after tests pass. |
Job Coverage, step Generate coverage on f7a3ba3 wanted to post the following: Framework coverageMutiple base shas were found when retrieving head reports: Modules coverageFluid properties
Full coverage reportsReports
Warnings
This comment will be updated on new commits. |
should be good now? |
doco recipe froze, no change in doco expected to break doco |
refs #29620