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

Potential Bug in Critical Pressure Estimation #2569

Open
mjohnson541 opened this issue Oct 30, 2023 · 2 comments
Open

Potential Bug in Critical Pressure Estimation #2569

mjohnson541 opened this issue Oct 30, 2023 · 2 comments
Labels
bug bug which will never be closed by the actions bot

Comments

@mjohnson541
Copy link
Contributor

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.

@rwest
Copy link
Member

rwest commented Nov 9, 2023

ReactionMechanismGenerator/RMG-database@22c2ec9

Copy link

github-actions bot commented Feb 7, 2024

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 github-actions bot added the stale stale issue/PR as determined by actions bot label Feb 7, 2024
@mjohnson541 mjohnson541 added bug bug which will never be closed by the actions bot and removed stale stale issue/PR as determined by actions bot labels Feb 7, 2024
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug bug which will never be closed by the actions bot
Projects
None yet
Development

No branches or pull requests

2 participants