Skip to content

Arkane: add check for iop(2/9=2000) in Gaussian load geometry#2758

Open
sevyharris wants to merge 2 commits intoReactionMechanismGenerator:mainfrom
sevyharris:iop29_warning
Open

Arkane: add check for iop(2/9=2000) in Gaussian load geometry#2758
sevyharris wants to merge 2 commits intoReactionMechanismGenerator:mainfrom
sevyharris:iop29_warning

Conversation

@sevyharris
Copy link
Copy Markdown
Contributor

Motivation or Problem

By default, Gaussian only prints Cartesians in the input orientation for small cases. Add a check for the IOP(2/9=2000) keyword so users don't get tripped up like I did here: #2757 with coordinates that don't match the Hessian.

Description of Changes

I added a check for the IOP(2/9=2000 keyword) when loading the geometry. If the molecule has 14+ atoms (this seems to be the threshold according to my experiments), this will raise a LogError if the keyword hasn't been specified.

Testing

I tested this using one Gaussian logfile that has the IOP and one that doesn't.
conformer_0002_iop.log
conformer_0002_no_iop.log

@sevyharris sevyharris linked an issue Feb 7, 2025 that may be closed by this pull request
@sevyharris
Copy link
Copy Markdown
Contributor Author

This is currently failing the unit tests because the toluene and benzyl Arkane examples have Gaussian logfiles with more than 14 atoms. I'm rerunning these calculations to see how much it changes the results. If it's a lot, I'll replace the logfiles. If it's not much, that means I've been overstating the error due to this bug and I may not fully understand the problem I thought I was diagnosing.

@github-actions
Copy link
Copy Markdown

This pull request 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 pull request, otherwise it will automatically be closed in 30 days.

@github-actions github-actions bot added the stale stale issue/PR as determined by actions bot label May 22, 2025
@github-actions github-actions bot added the abandoned abandoned issue/PR as determined by actions bot label Jun 22, 2025
@github-actions github-actions bot closed this Jun 22, 2025
@JacksonBurns
Copy link
Copy Markdown
Contributor

@sevyharris I see that this PR fell off - want to revive?

@sevyharris sevyharris removed stale stale issue/PR as determined by actions bot abandoned abandoned issue/PR as determined by actions bot labels Jun 23, 2025
@sevyharris sevyharris reopened this Jun 23, 2025
@github-actions
Copy link
Copy Markdown

This pull request 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 pull request, otherwise it will automatically be closed in 30 days.

@github-actions github-actions bot added the stale stale issue/PR as determined by actions bot label Sep 22, 2025
@sevyharris sevyharris removed the stale stale issue/PR as determined by actions bot label Sep 22, 2025
@github-actions
Copy link
Copy Markdown

This pull request 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 pull request, otherwise it will automatically be closed in 30 days.

@github-actions github-actions bot added the stale stale issue/PR as determined by actions bot label Dec 22, 2025
@github-actions github-actions bot added the abandoned abandoned issue/PR as determined by actions bot label Jan 21, 2026
@github-actions github-actions bot closed this Jan 21, 2026
@rwest rwest reopened this Apr 1, 2026
@rwest rwest removed stale stale issue/PR as determined by actions bot abandoned abandoned issue/PR as determined by actions bot labels Apr 1, 2026
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.

Projecting out rotors in Arkane leads to zero frequency

3 participants