Skip to content
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

New notebook + pyLIQTR update + QOL changes #58

Closed
wants to merge 4 commits into from
Closed

New notebook + pyLIQTR update + QOL changes #58

wants to merge 4 commits into from

Conversation

JonhasSC
Copy link
Collaborator

Adding new Quantum Chromodynamics notebook with the dynamics application + script for generating resource estimates for application 1 of that chapter. pyLIQTR updates as well are included

JonhasSC and others added 2 commits October 12, 2024 00:26
* wrote some down and added block of code for state prep

* wrote more on background of application and initial state

* finished writing background for qcd problem

* dynamics implemented for qcd

* cleaned up nb

* getting resource estimates now from qcd that actually includes correct neighbor interactions

* cleaned up qcd notebook

* cleaned up qcd notebook

* fixed bug for normalization factor

* made changes to random variables to grab random numbers from a normal gaussian distribution and changed evolution time to be sqrt(N) where N is the number of neutrinos

* modified grabbing params for hamiltonian generation

* adding script file to generate resource estimates for QCD

* modified gitignore to account for DS_STORE files

* tweaks to qcd script

* added files and section for analysis of resource estimates from performing a sweep

* Updated pylint badge

* Updated pylint badge

* improve readability and fix typos

---------

Co-authored-by: Jonhas Saul Colina <[email protected]>
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Zain Mughal <[email protected]>
… to not include disclaimer regarding WIP notebooks (#57)
Copy link
Collaborator

@zain2864 zain2864 left a comment

Choose a reason for hiding this comment

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

Final review done, looks good to merge to main!

@JonhasSC
Copy link
Collaborator Author

JonhasSC commented Nov 4, 2024

Closing PR as not appropriate to merge yet

@JonhasSC JonhasSC closed this Nov 4, 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.

2 participants