-
Notifications
You must be signed in to change notification settings - Fork 162
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
ASIrebase #1335
base: main
Are you sure you want to change the base?
ASIrebase #1335
Conversation
dc80e42
to
417d88e
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1335 +/- ##
==========================================
- Coverage 70.84% 70.83% -0.01%
==========================================
Files 231 233 +2
Lines 44186 44329 +143
==========================================
+ Hits 31303 31400 +97
- Misses 12883 12929 +46 ☔ View full report in Codecov by Sentry. |
b256742
to
96d636e
Compare
9953104
to
26c612b
Compare
Added electrostatic potential C API Added API functions for ASI API implemenation. Including import and export of Hamiltonian, Overlap and Density Matrices. Lock down ASI incompatible options when the interface is active (note, manual keyword required for changes in H/S due to limitations of ASI at the moment): * Forces can't be used if the hamiltonian is changed without matrix element derivatives. * SCC can't work unless the density matrix extraction is used to update the hamiltonian (no way of knowing at the moment). * Can't evaluate Mulliken populations if S changes at the moment. Note: imp_ham tester contains workaround for icx compiler 24.0 problem on returning bool. Co-authored-by: Ben Hourahine <benjamin.hourahine@strath.ac.uk>
@PavelStishenko this is now rebased on the current main (only two conflicts, so not so bad this time). Note that until there is a signaling mechanism for whether the overlap has been modified via ASI, it is unsafe to run SCC calculations. |
Contains changes from #1217 rebased on top of main at 1ab3e50, plus blocking of incompatible DFTB+ features if ASI is turned on.