Skip to content

Fix passing atomic numbers from PySCF to D4 interface#248

Merged
marvinfriede merged 2 commits intomainfrom
fix-pyscf-charges
Jul 28, 2024
Merged

Fix passing atomic numbers from PySCF to D4 interface#248
marvinfriede merged 2 commits intomainfrom
fix-pyscf-charges

Conversation

@marvinfriede
Copy link
Copy Markdown
Member

@marvinfriede marvinfriede commented Jul 10, 2024

@codecov
Copy link
Copy Markdown

codecov bot commented Jul 25, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 41.13%. Comparing base (d035dcd) to head (ef43883).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #248   +/-   ##
=======================================
  Coverage   41.13%   41.13%           
=======================================
  Files          33       33           
  Lines        2966     2966           
  Branches     1347     1347           
=======================================
  Hits         1220     1220           
  Misses       1005     1005           
  Partials      741      741           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@marvinfriede marvinfriede requested a review from awvwgk July 26, 2024 08:00
@awvwgk
Copy link
Copy Markdown
Member

awvwgk commented Jul 26, 2024

This PR contains many unrelated changes, please separate them into individual PRs which will be much quicker to review

@marvinfriede
Copy link
Copy Markdown
Member Author

This PR contains many unrelated changes, please separate them into individual PRs which will be much quicker to review

Everything is now separated. Merge #250 and #251 before this PR.

@marvinfriede marvinfriede requested a review from awvwgk July 28, 2024 12:03
@marvinfriede marvinfriede merged commit 4aa9e79 into main Jul 28, 2024
@marvinfriede marvinfriede deleted the fix-pyscf-charges branch July 28, 2024 14:52
@aizvorski aizvorski mentioned this pull request Aug 29, 2024
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.

PySCF interface retrieves incorrect atomic numbers if ECPs are used

2 participants