Skip to content

Pt kinetics#9

Open
kblondal wants to merge 13 commits intorwest:catfrom
kblondal:Pt_kinetics
Open

Pt kinetics#9
kblondal wants to merge 13 commits intorwest:catfrom
kblondal:Pt_kinetics

Conversation

@kblondal
Copy link
Copy Markdown

This is the Pt surface reaction library from the Deutschmann (2006) paper. A library from Vlachos (2007) will be added here as well.

Details include authors, DFT binding energy, linear scaling parameters, and 2D gas model comments.
Details include DFT binding energy, linear scaling parameters, and 2D gas model comments.
Added four species that were accidentally left out (entries with indices 84-87).
Added their labels in the tree.
@rwest
Copy link
Copy Markdown
Owner

rwest commented Dec 3, 2018

I did some tidying up of this using interactive rebase, and pushed it to my Pt_kinetics branch:
https://github.com/rwest/RMG-database/commits/Pt_kinetics

I could just merge it and close this, but as one of your purported goals is to learn how to make beautiful pull requests, I'll leave it, and let you see if you can make yours look a bit like it.

Things I did:

  • remove the commits that cancel each other out (eg. the ones that revert each other). (first I put them next to each other and merged them, to ensure they were exactly cancelling).
  • Put the one that fixes a mistake in Vlachos, right after the one that adds Vlachos, and merge them
  • Put the one that fixes a mistake in Deutschmann, right after the one that adds Deutschmann, and merge them
  • Edit the commit messages to make the first line shorter (then have a blank line).

rwest pushed a commit that referenced this pull request Jan 15, 2019
Commented-out training reaction #8 which now has a better rate estimate
(at combustion-relevant temperatures, the williams et al. study is for low T)
Removed training reaction #9 which is incorrect: NO2-2 (O atom has the
rad in this resonance structure) reacts with OH to form HOONO,
not HNO3 as stated in this reaction; this is also not in agreement
with the cited paper by Williams et al.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants