Skip to content

OPENQP Landau-Zener interface#225

Merged
JanosJiri merged 4 commits intomasterfrom
openqp-lz
Mar 17, 2026
Merged

OPENQP Landau-Zener interface#225
JanosJiri merged 4 commits intomasterfrom
openqp-lz

Conversation

@JanosJiri
Copy link
Contributor

@JanosJiri JanosJiri commented Mar 9, 2026

Interface to MRSF-TDDFT implemented in OPENQP. Currently only for Landau-Zener SH.

Created by Katarina Vosovicova (and JJ).

Created by Katarina Vosovicova and JJ
@JanosJiri JanosJiri requested a review from danielhollas March 9, 2026 11:44
@danielhollas
Copy link
Contributor

Cool! I'll have a look this week.

Copy link
Contributor

@danielhollas danielhollas left a comment

Choose a reason for hiding this comment

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

Great! Please see some minor suggestion below.

The main suggestion I have is to change the approach of generating the openqp input file. Instead of generating it inside the bash script, it would be simpler to have a template input file (e.g. openqp.inp.template) and use sed to only modify the parts that are variable during MD (e.g. path to geometry, path to guess, number of states etc).
The advantage would be that if the user wants to e.g. add some parameter to the input file, they can simply modify just the template file, instead of having to add the variable to the openqp.inp and then print it in the script. Does that make sense?

I think SHARK uses a similar strategy, and it's probably something we should do in other interfaces as well in the future.
Note that specifically for Surface Hopping we couldn't do that for Molpro since Molpro's input file is way more complicated and we do need to generate it on the fly.

@JanosJiri JanosJiri requested a review from danielhollas March 13, 2026 19:34
@JanosJiri
Copy link
Contributor Author

JanosJiri commented Mar 13, 2026

Thanks for all the suggestions, @danielhollas! Katka processed them all and created this new version. However, I don't understand why the tests fail since we uploaded only an interface; we did not touch the code at all.

@danielhollas
Copy link
Contributor

The failing test is unrelated I see it in other PRs as well, I'll investigate.

Co-authored-by: Daniel Hollas <danekhollas@gmail.com>
Copy link
Contributor

@danielhollas danielhollas left a comment

Choose a reason for hiding this comment

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

This is great, thanks!

@JanosJiri
Copy link
Contributor Author

JanosJiri commented Mar 17, 2026

Cool, I guess we should wait for #226 to be merged, right @danielhollas ?

@codecov
Copy link

codecov bot commented Mar 17, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.06%. Comparing base (1117c83) to head (6fe2ec6).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #225   +/-   ##
=======================================
  Coverage   84.06%   84.06%           
=======================================
  Files          47       47           
  Lines        6613     6613           
  Branches      766      766           
=======================================
  Hits         5559     5559           
  Misses        855      855           
  Partials      199      199           
Flag Coverage Δ
unittests 18.37% <ø> (ø)

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@danielhollas
Copy link
Contributor

Should be good now!

@JanosJiri JanosJiri merged commit 9e316d5 into master Mar 17, 2026
17 checks passed
@JanosJiri JanosJiri deleted the openqp-lz branch March 17, 2026 18:58
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.

2 participants