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

ONIOM results #917

Merged
merged 1 commit into from
Nov 28, 2023
Merged

ONIOM results #917

merged 1 commit into from
Nov 28, 2023

Conversation

Albkat
Copy link
Contributor

@Albkat Albkat commented Nov 27, 2023

add ONIOM specific property printout

Thanks @benbaed for the hint.

Signed-off-by: albert <92109627+Albkat@users.noreply.github.com>
@Albkat Albkat added the enhancement New feature or request label Nov 27, 2023
@@ -958,6 +958,9 @@ subroutine xtbMain(env, argParser)
type is(TxTBCalculator)
call write_energy(env%unit,res,fres, &
& (set%runtyp.eq.p_run_hess).or.(set%runtyp.eq.p_run_ohess).or.(set%runtyp.eq.p_run_bhess))
type is(TOniomCalculator)
call write_energy_oniom(env%unit,res,fres, &
& (set%runtyp.eq.p_run_hess).or.(set%runtyp.eq.p_run_ohess.or.(set%runtyp.eq.p_run_bhess)))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This has no effect on functionality of course, but is there a specific reason why you added the brackets like that?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nah, just a typo

@Albkat Albkat merged commit ddc8329 into grimme-lab:main Nov 28, 2023
11 checks passed
@Albkat Albkat deleted the ONIOM_res branch November 28, 2023 09:13
gorges97 pushed a commit to gorges97/xtb that referenced this pull request Mar 27, 2024
Signed-off-by: albert <92109627+Albkat@users.noreply.github.com>
Signed-off-by: Johannes Gorges <58849467+gorges97@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants