Skip to content
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

Draft: Closes #255. Hold NASA Cp constant when temp is out-of-bounds. #257

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

mgoodson-cvd
Copy link
Collaborator

Closes #255. When the temperature is out-of-bounds with the NASA polynomials, hold the Cp constant and adjust the enthalpy and entropy accordingly.

Note: this code was mostly authored by Alireza Mazaheri of NASA LaRC.

When the temperature is out-of-bounds with the NASA polynomials,
hold the Cp constant and adjust the enthalpy and entropy accordingly.

Note: this code was mostly authored by Alireza Mazaheri of NASA LaRC.

See mutationpp#255. Closes mutationpp#255.
Copy link

codecov bot commented Mar 7, 2024

Codecov Report

Attention: Patch coverage is 67.24138% with 19 lines in your changes are missing coverage. Please review.

Project coverage is 71.97%. Comparing base (aa01785) to head (6869155).

Files Patch % Lines
src/thermo/NasaDB.h 67.24% 19 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #257      +/-   ##
==========================================
+ Coverage   71.65%   71.97%   +0.31%     
==========================================
  Files         135      135              
  Lines        9089     9137      +48     
==========================================
+ Hits         6513     6576      +63     
+ Misses       2576     2561      -15     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Add Matthew Goodson and Alireza Mazaheri to the contributors list.

See mutationpp#255.
Update the version to 1.2.0.

See mutationpp#255.
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.

Out-of-bounds temperature treatment for NASA polynomials
2 participants