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

Non-deterministic choice between ring(Cyclopropene) and ring(oxirene) ? #2010

Open
rwest opened this issue Aug 19, 2020 · 2 comments
Open

Non-deterministic choice between ring(Cyclopropene) and ring(oxirene) ? #2010

rwest opened this issue Aug 19, 2020 · 2 comments
Labels
bug bug which will never be closed by the actions bot

Comments

@rwest
Copy link
Member

rwest commented Aug 19, 2020

Bug Description

Comments on several pull requests have noticed some non-identical thermo in the rmg/nitrogen test in the regression tests. It shows up something like this:

Non-identical thermo!
original:	O1[C]=N1
tested:	O1[C]=N1
Hf(300K)  |S(300K)   |Cp(300K)  |Cp(400K)  |Cp(500K)  |Cp(600K)  |Cp(800K)  |Cp(1000K) |Cp(1500K) 
    129.73|     51.79|      9.14|      9.59|      9.99|     10.34|     10.90|     11.38|     12.40
    154.90|     56.55|      9.79|      9.15|      8.58|      8.33|      9.02|     10.26|     10.96
thermo: Thermo group additivity estimation: missing(O2s-CdN3d) + group(N3s-CsHH) + group(Cds-CdsCsCs) + ring(Cyclopropene) + radical(Cds_P)
thermo: Thermo group additivity estimation: missing(O2s-CdN3d) + group(N3s-CsHH) + group(Cds-CdsCsCs) + ring(oxirene) + radical(Cds_P)

It appears in comments on these separate pull requests:

The diff may appear in other pull requests but without anyone delving into the logs and commenting.

It is mostly just mentioned and then ignored, although @goldmanm discussed it briefly here

One species had different thermo [C]1=NO1, which was due to whether RMG used the group additivity value of ring(Cyclopropene) or ring(oxirene), a 30 kcal difference. Neither of these groups is likely giving accurate values for the thermo, but the change either shows some indeterminate nature of RMG or that we are losing resonance structures in the simulation with these changes.

I suspect it's an indeterminate behaviour.

How To Reproduce

Look at the logs of the automated regression tests, or (presumably) estimate the thermo of O1[C]=N1 a bunch of times.

Expected Behavior

Thermo of O1[C]=N1 would be consistent.

Installation Information

Describe your installation method and system information.

  • detected by reading regression logs, so whatever is in Travis
@github-actions
Copy link

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.

@rwest
Copy link
Member Author

rwest commented Jun 22, 2023

This is still a problem (see #2490 (comment) )

@rwest rwest added bug bug which will never be closed by the actions bot and removed stale stale issue/PR as determined by actions bot labels Jun 22, 2023
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

1 participant