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

Error in R1234ze Saturation Density #86

Open
chmarti1 opened this issue Jan 8, 2024 · 0 comments
Open

Error in R1234ze Saturation Density #86

chmarti1 opened this issue Jan 8, 2024 · 0 comments
Labels

Comments

@chmarti1
Copy link
Owner

chmarti1 commented Jan 8, 2024

In version 2.2.4, there is an error in the saturated liquid density model for R1234ze (mp.C3H2F4_1). One of the empirical coefficients was incorrectly entered. The model still passes its integrity checks recommended by the original paper, but the saturation property calculations are not correct. This problem has already been fixed in the planned release of version 2.2.5.

If you want to correct it yourself in your own release, open
/your/pyromat/installation/dir/data/mp/C3_H2_F4_1.hpd
Line 96 reads 1.9696
It should read 2.2456

@chmarti1 chmarti1 added the bug label Jan 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant