-
Notifications
You must be signed in to change notification settings - Fork 14
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
Example for determinants #167
Comments
I think it would make sense to include a complete example (including the call to CASSCF/CASCI module of PySCF) and not the truncated part. Or even better - show how to write the HF wave function first and the CI one second so that we have a complete pipeline. Then we can do the same for QP2 so that the users have concrete use cases for CI I/O in two different codes. |
@q-posev Totally agree but I think it would make more sense to register the mcscf object is not it ? I am trying to extract data which make sense between mcscf object and trexio. In the meanwhile I can create this two examples! |
Indeed, I think what @scemama meant is adding an example using low-level PySCF functionalities like you did in |
@scemama @NastaMauger we have an example of determinants I/O in the dedicated |
Here are my scripts, which export all necessary information from RHF to trexio and also create a det.hdf5 file using specific functions. Does this meet your requirements regarding the example documentation? PS: Both script are compatible with the 0-based indices and the electron group infos. |
Thank you for sharing the scripts @NastaMauger ! For the pedagogical purposes, it might be better to have your script converted into a Jupyter notebook. And I noticed that some functions are outdated (e.g. On the other hand, not sure if we want to keep the call to |
@q-posev Sorry, I'm getting confused between trexio, trexio-tools and pyscf-forge/trexio Let me know if it suits your requirement |
@NastaMauger Yes, I am getting lost with all the discussions in parallel too :-) Then we will put it in the |
@q-posev The PR has been merged. Do the two examples attached to the PR description meet your requirements for the example you would like to include in this repo? |
Thank you @NastaMauger for your very important contribution on this! Would it be possible to make your two scripts from the PR into one script with a conditional if/else block to choose between FCI and selected CI? if do_fci:
<your code for pyscf FCI preparation and kernel here>
elif do_sci:
<your code for pyscf selected CI preparation and kernel here> This way it will be easier to convert into a Jupyter notebook. Would you mind creating a PR in the trexio-tutorials repo and not here? Once we have a notebook there - I will make necessary changes so that TREXIO documentation webpage is synced with trexio-tutorials. Thanks! |
@q-posev Sure. I have never done jupyter stuff. Let me check that so I can do a clean notebook for these new features ! |
@NastaMauger thank you! Don't worry about jupyter, if you only have a working python script with the aforementioned changes - i can transform it into a notebook myself. We store notebooks in the markdown format and not in the conventional ipynb format because the former is git-friendly, unlike the latter. There is a section in trexio-tutorials README about that if you are interested. |
Hello,
I think it might also be beneficial for users to have a direct example showing how to extract and register the determinants from a PySCF script.
For instance, I not only needed to confirm the 1-based indexing for the determinant list, but also to ensure that my TREXIO file contained the correct number of states (which, to be honest, was not obvious in my case since I am working with the ground state, and this section in the TREXIO documentation pertains to excited states).
The example determinant file you already have does not mention this and might not provide enough detail on how to do this from PySCF. Additionally, converting to a bit-field format may not be obvious depending on the user's level of knowledge. I believe you could directly use my PySCF script (which is highly inspired from your ResultsFIle work) related to this part, which is attached.
Best
Originally posted by @NastaMauger in TREX-CoE/trexio_tools#45 (comment)
The text was updated successfully, but these errors were encountered: