-
Notifications
You must be signed in to change notification settings - Fork 586
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
Extending support for larger atomic numbers #5821
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5821 +/- ##
=========================================
Coverage ? 99.65%
=========================================
Files ? 427
Lines ? 40728
Branches ? 0
=========================================
Hits ? 40589
Misses ? 139
Partials ? 0 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @obliviateandsurrender, I only have a couple of non-blocking comments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me @obliviateandsurrender
…AI/pennylane into atomic-numbers-upgrade
…o atomic-numbers-upgrade
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @obliviateandsurrender, left a few comments.
…o atomic-numbers-upgrade
…AI/pennylane into atomic-numbers-upgrade
Context: Currently, we support only first- or second-row elements of the periodic table for molecule and Hamiltonian constructions. This PR aims to push this limit to all the elements.
Description of the Change: Updates the
atomic_numbers
dictionary in theqchem/basis_data.py
and adapts the qchem functionality with this update.Benefits: We can support more elements by letting users use
load_data=True
keyword argument.Possible Drawbacks: None
Related GitHub Issues: