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

Update atom enthalpies for H, C, N, O, F, Cl, Br, I with data from ATcT #416

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

cgrambow
Copy link

@cgrambow cgrambow commented May 15, 2020

I'm going to get this PR going. I used updated data from ATcT. Some things I noticed:

  • The previous data for Br was horribly wrong. New data should be double-checked.
  • ATcT do not have S data. Where do we get this from? I double-checked that S corresponds to the data in NIST-JANAF.
  • Are there new sources for the other atoms/elements?

@dranasinghe, I assigned you to this PR. None of the existing BACs have been corrected yet based on the updated data.

Let's discuss.

RMG-Py PR: ReactionMechanismGenerator/RMG-Py#1955

@cgrambow cgrambow requested a review from amarkpayne May 15, 2020 21:00
@cgrambow cgrambow marked this pull request as draft May 15, 2020 21:00
@cgrambow cgrambow force-pushed the update_atom_enthalpies branch from e7ac70e to 5719e06 Compare May 15, 2020 21:23
@cgrambow cgrambow mentioned this pull request May 16, 2020
@cgrambow cgrambow force-pushed the update_atom_enthalpies branch from 5719e06 to 179e6c2 Compare May 19, 2020 22:53
@cgrambow cgrambow marked this pull request as ready for review May 19, 2020 22:55
@cgrambow cgrambow mentioned this pull request May 19, 2020
Copy link

@dranasinghe dranasinghe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bill suggested we need to make sure we have the best possible numbers. atom_hf numbers are correct. The problem is atom_thermal. Give me couple of days to make sure we have the correct numbers.

'Li': 1.1, 'Be': 0.46, 'B': 0.29, 'C': 0.25, 'N': 1.04, 'O': 1.04, 'F': 1.05, 'Ne': 1.481,
'Na': 1.54, 'Mg': 1.19, 'Al': 1.08, 'Si': 0.76, 'P': 1.28, 'S': 1.05, 'Cl': 1.1,
'K': 1.481, 'Ca': 1.481, 'Ti': 1.802, 'Cu': 1.481, 'Zn': 1.481, 'Ge': 1.768, 'Br': 1.481,
atom_thermal = {'H': 1.012, 'He': 1.481,

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is a problem with how H( 0 k) and H(298K) presented, each uses their own standard state. So it Looks like I have to email ATCT and request thermal correction relative to H(298 K)

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aren't atom_thermal just ideal gas corrections? They shouldn't change, right?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think they depend on Cp (int Cp=0 to Cp 298)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's make sure.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, they're the enthalpy increment of the elements (or 1/2 for diatomic reference states). Translational and rotational contributions are constant. If the vibrational frequencies of diatomics have changed in recent times, then the vibrational contribution could have changed, and if the excited state energy levels have changed in recent times, then the electronic contribution could have changed. My assumption would be that the frequencies and energy levels of diatomics are very well known, so these numbers should be accurate. I'll try to find recent frequencies and energy levels to calculate some of these numbers and see if my calculations agree.

Copy link

@dranasinghe dranasinghe May 20, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We do 2 for loops in [RMG-Py/arkane/encorr/corr.py](https://github.com/ReactionMechanismGenerator/RMG-Py/blob/74cee80a8c688b04dba265b2bb0b2a10724bee41/arkane/encorr/corr.py#L140) line 120 to 140. I Don't see a problem replacing with one for loop and use the J as units. 

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, it'll be exactly the same combining them into one number. What I'm trying to say is that it's better having them separate in the database because it makes for easier comparison with literature. For example, ATcT only lists atom_hf so a direct comparison to ATcT wouldn't be easy if we combine the numbers in the database.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree with your comment that it would be easy to compare but the way we use/code we would never able to use that data. So why not combine?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The way that Arkane separates all the terms (and then adds them back together) is the standard way to compute thermochemistry in literature, so I think we should keep it the way it is.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

By the way, I calculated the thermal corrections for all the gas phase diatomics, and my calculations agree with the values in the database.

@rwest rwest force-pushed the update_atom_enthalpies branch from 795d984 to 3dd4b92 Compare July 15, 2023 19:18
@rwest rwest changed the base branch from master to main July 15, 2023 19:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants