RMG-Py icon indicating copy to clipboard operation
RMG-Py copied to clipboard

Potential Bug in Critical Pressure Estimation

Open mjohnson541 opened this issue 1 year ago • 2 comments

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)) 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.

mjohnson541 avatar Oct 30 '23 19:10 mjohnson541

https://github.com/ReactionMechanismGenerator/RMG-database/commit/22c2ec91f2be4ca5d878034f3f0df6f14b958c7e

rwest avatar Nov 09 '23 04:11 rwest

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.

github-actions[bot] avatar Feb 07 '24 08:02 github-actions[bot]