updated DFT_QCI_thermo with uncertainty data from paper#49
updated DFT_QCI_thermo with uncertainty data from paper#49nickvandewiele wants to merge 2 commits into
Conversation
also: updated indeces > 33 to comply with original indices of paper.
|
This pull request solves ReactionMechanismGenerator/RMG-Py#267 The script to do this can be re-used: |
|
Looks good, thanks for doing this. However, I do wonder what we should do about the uncertainties that are reported as zero. Presumably @cfgold calculated that they were less than 0.05 and he was reporting things to 1 decimal place so rounded to 0.0 instead of 0.1. However, if one day this gets used in some optimization or "consistent set" checking tool (as we saw many of at the Symposium), then an uncertainty of precisely zero will make the constraints hard to satisfy! Should we put 0.05? |
|
Agreed. I will update all 0.0s to 0.05s. |
increment version to v0.3.0
also: updated indeces > 33 to comply with original indices of paper.