-
Notifications
You must be signed in to change notification settings - Fork 136
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
Inconsistent electronic states for atomic carbon #171
Comments
This was referenced Mar 18, 2017
This was referenced Mar 21, 2017
alongd
added a commit
that referenced
this issue
Mar 24, 2017
Originally I mistakenly naked these libraries `0.2`, where infact they are version `2.0`. The libraries names were corrected now as well. Main changes in this commit are: - The AdjLists of [C] were corrected to a multiplicity 3 instead of 5 (see issue #171) - The AdjLists of [CH] were corrected to a multiplicity 2 instead of 4 (see issue #172) - THe representation of CH2 and CH2(S) was correct and wasn't changed
alongd
added a commit
that referenced
this issue
Mar 24, 2017
Originally I mistakenly naked these libraries `0.2`, where infact they are version `2.0`. The libraries names were corrected now as well. Main changes in this commit are: - The AdjLists of [C] were corrected to a multiplicity 3 instead of 5 (see issue #171) - The AdjLists of [CH] were corrected to a multiplicity 2 instead of 4 (see issue #172) - The representation of CH2 and CH2(S) was correct and wasn't changed - Reactions of `[CH]*` and `[OH]*` (which currently cannot be represented in RMG) were added as a comment
alongd
added a commit
that referenced
this issue
Mar 24, 2017
Originally I mistakenly named these libraries `0.2`, where infact they are version `2.0`. The libraries names were corrected now as well. Main changes in this commit are: - The AdjLists of [C] were corrected to a multiplicity 3 instead of 5 (see issue #171) - The AdjLists of [CH] were corrected to a multiplicity 2 instead of 4 (see issue #172) - The representation of CH2 and CH2(S) was correct and wasn't changed - Reactions of `[CH]*` and `[OH]*` (which currently cannot be represented in RMG) were added as a comment
I think this can be closed after #180 is merged. |
Thanks Alon! |
fixed by #180 |
mliu49
pushed a commit
to mliu49/RMG-database
that referenced
this issue
Mar 29, 2017
Originally I mistakenly named these libraries `0.2`, where infact they are version `2.0`. The libraries names were corrected now as well. Main changes in this commit are: - The AdjLists of [C] were corrected to a multiplicity 3 instead of 5 (see issue ReactionMechanismGenerator#171) - The AdjLists of [CH] were corrected to a multiplicity 2 instead of 4 (see issue ReactionMechanismGenerator#172) - The representation of CH2 and CH2(S) was correct and wasn't changed - Reactions of `[CH]*` and `[OH]*` (which currently cannot be represented in RMG) were added as a comment
alongd
added a commit
to alongd/RMG-database
that referenced
this issue
Aug 23, 2017
Originally I mistakenly named these libraries `0.2`, where infact they are version `2.0`. The libraries names were corrected now as well. Main changes in this commit are: - The AdjLists of [C] were corrected to a multiplicity 3 instead of 5 (see issue ReactionMechanismGenerator#171) - The AdjLists of [CH] were corrected to a multiplicity 2 instead of 4 (see issue ReactionMechanismGenerator#172) - The representation of CH2 and CH2(S) was correct and wasn't changed - Reactions of `[CH]*` and `[OH]*` (which currently cannot be represented in RMG) were added as a comment
goldmanm
pushed a commit
that referenced
this issue
Aug 24, 2017
Originally I mistakenly named these libraries `0.2`, where infact they are version `2.0`. The libraries names were corrected now as well. Main changes in this commit are: - The AdjLists of [C] were corrected to a multiplicity 3 instead of 5 (see issue #171) - The AdjLists of [CH] were corrected to a multiplicity 2 instead of 4 (see issue #172) - The representation of CH2 and CH2(S) was correct and wasn't changed - Reactions of `[CH]*` and `[OH]*` (which currently cannot be represented in RMG) were added as a comment
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Running
grep '1 *C *u[0-9] *p[0-9] *c[0-9] *$' -B4 -A1 -r .
in the database shows a range of choices that have been made for the multiplicity and electronic structure of atomic carbon. In a couple of cases this has obviously been thought about , for exampleprimaryThermoLibrary.py
contains bothand
(the chosen labels make it clear that one was meant to be singlet and one triplet)
however, some of them appear to be auto-generated adjacency lists in the quintuplet state,
which is probably not right.
We should check each of these:
(Also, the SMILES interpretation and generation is broken for these, but that's an RMG-Py issue that I'll open later)
Check these off when checked (or corrected):
The text was updated successfully, but these errors were encountered: