Skip to content

SH: ignore_state for MOLPRO-SH#132

Merged
danielhollas merged 2 commits intomasterfrom
sh-ignore-state
May 19, 2022
Merged

SH: ignore_state for MOLPRO-SH#132
danielhollas merged 2 commits intomasterfrom
sh-ignore-state

Conversation

@danielhollas
Copy link
Copy Markdown
Contributor

@danielhollas danielhollas commented May 9, 2022

ignore_state SH option is useful e.g. to ignore ground state when running TDDFT dynamics. Currently I am using it to ignore the third state in the ethylene SA3-CAS(2,2) dynamics. For some reason, I originally implemented this option only in the tera SH interface. Here I move it so it is available to MOLPRO-SH as well.

TODO: write a test for this functionality.
TODO: Check that initial_state /= ignore_state

@danielhollas danielhollas self-assigned this May 9, 2022
@codecov
Copy link
Copy Markdown

codecov Bot commented May 9, 2022

Codecov Report

Merging #132 (997695c) into master (dedb7f1) will increase coverage by 0.05%.
The diff coverage is 97.05%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #132      +/-   ##
==========================================
+ Coverage   94.32%   94.38%   +0.05%     
==========================================
  Files          42       42              
  Lines        5864     5873       +9     
==========================================
+ Hits         5531     5543      +12     
+ Misses        333      330       -3     
Flag Coverage Δ
unittests 25.84% <0.00%> (-0.07%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/force_terash.F90 95.89% <ø> (+1.04%) ⬆️
src/surfacehop.F90 95.07% <96.96%> (+0.14%) ⬆️
src/abin.F90 91.47% <100.00%> (ø)

@danielhollas danielhollas marked this pull request as ready for review May 13, 2022 11:14
@danielhollas danielhollas requested a review from suchanj May 13, 2022 11:14
@danielhollas danielhollas merged commit d3f7483 into master May 19, 2022
@danielhollas danielhollas deleted the sh-ignore-state branch May 19, 2022 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants