[pull] master from MRChemSoft:master #92
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Close #419 We now use qcelemental to fetch Bondi radii, and use alpha=1.2 for hydrogen --------- Co-authored-by: Roberto Di Remigio Eikås <robertodr@users.noreply.github.com>
* cavity function used directly instead of projected first * clear d_V * Update SCRF.cpp define ranks in accelerator, so that all mpi ranks use kain * cavity function used directly instead of projected first * clear d_V * Update SCRF.cpp define ranks in accelerator, so that all mpi ranks use kain * manage to pull down to 12.2 and 30.0 gb * reduced memory usage to ca. 3 gb from 8 gb * Remove print statement for memory * Small changes to desctructor and clear * Use latest MRCPP * Use default initialization in header file * Do some cleaning after feedback * Remove ´optimizer´ option from inputs * Fix tests --------- Co-authored-by: Roberto Di Remigio Eikås <robertodr@users.noreply.github.com> Co-authored-by: Gabrielgerez <gsa017@uit.no>
* Convert default radii to bohr when input units are not angstrom * Update template.yml (formatting and documentation) * Refine fix for unit conversion of cavity params, add tests
* Refactor the Permittivity function for ease of future implementation work. * Add ShiftFunction to CMakeLists * Add virtual and override where necessary * Remove evalf definition from shiftFunction * Rename ShiftFunction to StepFunction * Update src/environment/Permittivity.cpp Co-authored-by: Roberto Di Remigio Eikås <robertodr@users.noreply.github.com> * Update src/environment/StepFunction.cpp Co-authored-by: Roberto Di Remigio Eikås <robertodr@users.noreply.github.com> * Update src/environment/StepFunction.cpp Co-authored-by: Roberto Di Remigio Eikås <robertodr@users.noreply.github.com> * Update src/environment/StepFunction.cpp Co-authored-by: Roberto Di Remigio Eikås <robertodr@users.noreply.github.com> * Update src/environment/StepFunction.h Co-authored-by: Roberto Di Remigio Eikås <robertodr@users.noreply.github.com> * Update src/environment/StepFunction.h Co-authored-by: Roberto Di Remigio Eikås <robertodr@users.noreply.github.com> * Pass cavity as a shared_ptr to save memory. * Fix rebase error * Remove use for `flipFunction` in Permittivity * Update src/environment/Permittivity.h Co-authored-by: Roberto Di Remigio Eikås <robertodr@users.noreply.github.com> * Update src/environment/StepFunction.cpp Co-authored-by: Roberto Di Remigio Eikås <robertodr@users.noreply.github.com> * Add print_header to detail namespace * Update src/environment/StepFunction.h Co-authored-by: Roberto Di Remigio Eikås <robertodr@users.noreply.github.com> --------- Co-authored-by: Roberto Di Remigio Eikås <robertodr@users.noreply.github.com>
* Use C++17
* Use std::tie to unpack tuple return value
* Remove some cruft from Cavity and Permittivity
* SCRF deals in densities only, not orbitals
* Refactor Reaction{Potential,Operator} following Coulomb example
In preparation for the computation of the response terms, we make ReactionPotential abstract and add
subclass ReactionPotentialD1 to deal with one single set of orbitals.
* Modify input to accept static and dynamic permittivities
* Handle nonequilibrium solvation more gracefully
* Sign change was necessary
* Update input parser
* Small fixes to include order
* Rename section for permittivity outside
* Add tests
Small fix to ease compilation under MAC
needs to be close to the class definition
Removed the unused get_my_chunk function from orbital_utils.cpp.
More consistent numerical results and repaired exchange
Add missing template parameters
* first version two electron integrals * print times for two electron integrals * bug shared potential (code hangs) * 2 el integrals test and change indices order * add two electron integrals test file * remove obsolete rescaling for Zora (taken care by mrcpp now) * update test results for new mrcpp (default for crop is absprec=true) * more test results updates for new mrcpp (default for crop is absprec=true) * lower test threshold for xc_hessian_pbe * Update tests/qmoperators/two_el_int.cpp Co-authored-by: Roberto Di Remigio Eikås <robertodr@users.noreply.github.com> --------- Co-authored-by: Luca Frediani <luca.frediani@gmail.com> Co-authored-by: gitpeterwind <peterw@met.no> Co-authored-by: Roberto Di Remigio Eikås <robertodr@users.noreply.github.com>
* rebased master; fixed some last warnings * fetched wrong MRCPP
* initial conversion of occupancies from integer to double * Initial implementation of core hole calculations * Switching logic so that restricted checks depend on doubly occ (only happens for rest) not singly occ (which could be unrest or rest with a hole) * XX with occupancies * updating input file for explicit occupancies * fixing occupancies/spin factors for XX in restricted calculations * removing unneeded print statement * adding positions of the orbitals to enable association with a particular atomic orbital * first try * restricted MOM * first try for IMOM; only IMOM in this version * first cleanup, code now readable, but longer * small bugfix * added input in input parser for MOM/IMOM * small bugfix * new logic to treat different occupation numbers; cleanup * small formatting * small temporary fix in restricted occupations * print orbital occupations as doubles instead of integers * for now only removal of one electron possible# * small bugfix in ExchangePotentialD2.cpp * bugfix in initial guess * IMOM: fetch Orbitals before calculation of initial guess energy; some general cleanup; added first test for DeltaSCF * small bugfix in test * small bugfix in driver.cpp and some changes to the core-hole test * added tests * changing copies to deep_copies * some cleanup + some more output depending on print level * - Implemented partial localization for different occupations - Fixed values for tests - some cleanup * small bugfix for unrestricted MOM/IMOM calculations * remove partial localization * added tests for unrestricted IMOM; some cleanup * small cleanup * incorporated PR comments * first try * first try for IMOM; only IMOM in this version * first cleanup, code now readable, but longer * added input in input parser for MOM/IMOM * new logic to treat different occupation numbers; cleanup * small formatting * IMOM: fetch Orbitals before calculation of initial guess energy; some general cleanup; added first test for DeltaSCF * small bugfix in test * small bugfix in driver.cpp and some changes to the core-hole test * added tests * changing copies to deep_copies * some cleanup + some more output depending on print level * - Implemented partial localization for different occupations - Fixed values for tests - some cleanup * small bugfix for unrestricted MOM/IMOM calculations * remove partial localization * added tests for unrestricted IMOM; some cleanup * small cleanup * make code mostly compilable * cleanup * fixed bug in calculation of number of electrons that lead to failing tests * updated test references for core-hole and IMOM tests * fixed MPI print issues, split tests to avoid timeout * added missing files * updated tests * storing mom orbitals in the Molecule object --------- Co-authored-by: Laura Ratcliff <laura_ratcliff@hotmail.com> Co-authored-by: Niklas Göllmann <niklas@tromso-eksterne-763.wifi.uit.no> Co-authored-by: Niklas Göllmann <niklas@tromso-eksterne-2394.wifi.uit.no>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
9 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )