MRChemSoft / mrchem

MultiResolution Chemistry
GNU Lesser General Public License v3.0
27 stars 21 forks source link

Enabling `sad_dz` initial guess for the 5th row elements #399

Closed Brakjen closed 2 years ago

Brakjen commented 2 years ago

I have added SAD density and basis set files for the 5th row elements.

The script sad_row5.py automatically generates the files for the 5th row.

codecov[bot] commented 2 years ago

Codecov Report

Merging #399 (3ed928b) into master (7d64a92) will not change coverage. The diff coverage is n/a.

:exclamation: Current head 3ed928b differs from pull request most recent head 8141f2e. Consider uploading reports for the commit 8141f2e to get more accurate results

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #399   +/-   ##
=======================================
  Coverage   68.51%   68.51%           
=======================================
  Files         179      179           
  Lines       14300    14300           
=======================================
  Hits         9798     9798           
  Misses       4502     4502           

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 7d64a92...8141f2e. Read the comment docs.

stigrj commented 2 years ago

How much work to extend to a full-fledged molecular GTO guess from PySCF? We would then need to print the MO matrix instead of the D matrix (easy?), and we need to build a full basis set file in Dalton format (a bit more annoying?).

stigrj commented 2 years ago

Actually, the Dalton basis file generator can be fetched from here 8b43d3505dfa94ec6f7a95fb81e77193ee83621f, we just need steal a full basis set file from the dalton source for a particular basis set.

stigrj commented 2 years ago

There's still a merge in the end :thinking: And now there's an extra commit changing the Betzy example, should this be there?

Brakjen commented 2 years ago

There's still a merge in the end 🤔 And now there's an extra commit changing the Betzy example, should this be there?

That one file has been marked as "modified" on my master for a long time. I don't know where it came from. I commited it to get rid of it...