-
Notifications
You must be signed in to change notification settings - Fork 233
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
Potential Bug in Critical Pressure Estimation #2569
Closed
mjohnson541 opened this issue
Oct 30, 2023
· 2 comments
· Fixed by ReactionMechanismGenerator/RMG-database#636
Closed
Potential Bug in Critical Pressure Estimation #2569
mjohnson541 opened this issue
Oct 30, 2023
· 2 comments
· Fixed by ReactionMechanismGenerator/RMG-database#636
Labels
bug
bug which will never be closed by the actions bot
Comments
This issue is being automatically marked as stale because it has not received any interaction in the last 90 days. Please leave a comment if this is still a relevant issue, otherwise it will automatically be closed in 30 days. |
JacksonBurns
added a commit
to ReactionMechanismGenerator/RMG-database
that referenced
this issue
Feb 8, 2024
JacksonBurns
added a commit
to ReactionMechanismGenerator/RMG-database
that referenced
this issue
Mar 8, 2024
JacksonBurns
added a commit
to ReactionMechanismGenerator/RMG-database
that referenced
this issue
Mar 8, 2024
jonwzheng
pushed a commit
to ReactionMechanismGenerator/RMG-database
that referenced
this issue
Jun 12, 2024
NellyMitnik
pushed a commit
to ReactionMechanismGenerator/RMG-database
that referenced
this issue
Dec 25, 2024
NellyMitnik
pushed a commit
to ReactionMechanismGenerator/RMG-database
that referenced
this issue
Dec 25, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I was contacted and asked about a potential issue with the critical pressure estimation in RMG. Critical pressure is computed using Joback method described:
Joback & Reid (1987): (https://doi.org/10.1080/00986448708960487[)](https://doi.org/10.1080/00986448708960487))
Joback’s thesis https://dspace.mit.edu/handle/1721.1/15374
They suggested that RMG is using the equations from the thesis and the groups from the paper, which is a problem for critical pressure because it uses different sign conventions for the group additive term (in the thesis the group terms are added, in the paper they are subtracted).
I took a cursory look at this and it looks like they are correct. I checked the history this system has been touched a few times making me a bit surprised this difference hasn't been noticed. In theory it can be fixed by simply changing the sign in the equation to negative.
The text was updated successfully, but these errors were encountered: