-
Notifications
You must be signed in to change notification settings - Fork 205
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
QCBM algorithm #271
Conversation
@GenericP3rson Can you help me with this |
Hi, don’t worry about getting that working, you just need to pass the functional tests! |
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.
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!
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.
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.)
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]>
@GenericP3rson what do you think about formatting now? |
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.
Good to go!
Purpose
Added Quantum Circuit Born Machine (QCBM) algorithm.
Details
torchquantum/algorithms
.Mentions
@GenericP3rson