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

QCBM algorithm #271

Merged
merged 11 commits into from
Jun 12, 2024
Merged

QCBM algorithm #271

merged 11 commits into from
Jun 12, 2024

Conversation

Gopal-Dahale
Copy link
Contributor

Purpose

Added Quantum Circuit Born Machine (QCBM) algorithm.

Details

Mentions

@GenericP3rson

@Gopal-Dahale
Copy link
Contributor Author

@GenericP3rson Can you help me with this flake8 error in github actions.

@01110011011101010110010001101111
Copy link
Collaborator

Hi, don’t worry about getting that working, you just need to pass the functional tests!

Copy link
Collaborator

Choose a reason for hiding this comment

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

Great work so far! Main thing to change is formatting, see comments below. Thanks for the great work and feel free to reach out with questions!

examples/QCBM/README.md Show resolved Hide resolved
examples/QCBM/qcbm_gaussian_mixture.py Outdated Show resolved Hide resolved
Copy link
Collaborator

Choose a reason for hiding this comment

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

Great work! A few last minor changes. If you could just keep the tabbing consistent in the qcbm.py file, that would be great! (A few examples are suggested—note that there might be some oddities with the spaces converting properly into tabs. I think it would be most optimal to just double-check that everything is consistent in the file itself.)

torchquantum/algorithm/qcbm.py Outdated Show resolved Hide resolved
torchquantum/algorithm/qcbm.py Outdated Show resolved Hide resolved
torchquantum/algorithm/qcbm.py Show resolved Hide resolved
torchquantum/algorithm/qcbm.py Outdated Show resolved Hide resolved
torchquantum/algorithm/qcbm.py Outdated Show resolved Hide resolved
Gopal-Dahale and others added 6 commits June 6, 2024 21:22
Co-authored-by: GenericP3rson <[email protected]>
Co-authored-by: GenericP3rson <[email protected]>
Co-authored-by: GenericP3rson <[email protected]>
Co-authored-by: GenericP3rson <[email protected]>
Co-authored-by: GenericP3rson <[email protected]>
@Gopal-Dahale
Copy link
Contributor Author

@GenericP3rson what do you think about formatting now?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Good to go!

@01110011011101010110010001101111 01110011011101010110010001101111 changed the base branch from main to unitary-hack June 11, 2024 17:00
@Hanrui-Wang Hanrui-Wang merged commit 60ace1e into mit-han-lab:unitary-hack Jun 12, 2024
5 of 7 checks passed
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.

Adding new examples and algorithms for TQ
3 participants