Skip to content

Conversation

@marwafar
Copy link
Collaborator

@marwafar marwafar commented Nov 5, 2025

In this PR:
We added UHF examples in 'molecular_hamiltonians.rst', 'generate_molecular_hamiltonians.py'

This part of the doc covers the new update of UHF in #309

Signed-off-by: marwafar <[email protected]>
@copy-pr-bot
Copy link

copy-pr-bot bot commented Nov 5, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

Signed-off-by: marwafar <[email protected]>
Signed-off-by: marwafar <[email protected]>
@marwafar marwafar marked this pull request as ready for review November 7, 2025 02:46
@bmhowe23 bmhowe23 added the documentation Improvements or additions to documentation label Nov 7, 2025
Copy link
Collaborator

@mawolf2023 mawolf2023 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have thoughts on general organization, but will save those for later as discussed with Marwa.

0,
nele_cas=2,
norb_cas=3,
UR=True,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there any merit to just calling this option "UHF" rather than "UR"? May just be easier to remember.

Note that we use the same API but,toggle `MP2=True` and `integrals_natorb=True`.
Note that we use the same API but,toggle `MP2=True` and `integrals_natorb=True`. This will generate the molecular orbitals from MP2 natural orbitals, and compute the Hamiltonian integrals in this basis.
This option is unallowed yet when using `UR=True`. When using `UR=True`, only UHF molecular orbitals are employed to generate the electronic Hamiltonian.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"not yet available for"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants