-
Notifications
You must be signed in to change notification settings - Fork 52
This issue was moved to a discussion.
You can continue the conversation there. Go to discussion →
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
Neon with quantum-corrected PR #208
Comments
There wasn't a particular reason. I think we were just benchmarking and picked one set of parameters. You can specify your own set of parameters using the |
I guess my thought was if you are going to pick one it would make sense to follow their model selection to match the defaults in ThermoPack (what I am comparing against as we speak). I added this model to teqp a couple of days ago. |
We just benchmarked against the paper at the time since there was no thermopack back then. I am wondering why the empirical model wasn't the best for all cases? Since the FH1 and FH2 are just further-constrained versions of the empirical model. |
I agree I had the same question |
This is more of a discussion than an issue; I'll just convert it |
This issue was moved to a discussion.
You can continue the conversation there. Go to discussion →
In the paper of Aasen they select the FH1 model for neon, but Clapeyron uses the empirical parameters instead. Is there a reason why? Thermopack seems to match the values in the paper
The text was updated successfully, but these errors were encountered: