-
Notifications
You must be signed in to change notification settings - Fork 204
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Sorry. For some reason, an already merged commit showed up here. Second, should I list the changes I made in the comments? It tends to format oddly outside of Jupyter and would be a bit of a timesaver. |
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.
Thanks! Most changes are great, I've just left a couple of comments.
Co-authored-by: Frank Harkins <[email protected]>
Co-authored-by: Frank Harkins <[email protected]>
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.
Looks good, thanks!
Tests are failing but I'm merging as they're not related to this PR, and fixing them is going to be very involved. |
the underlying graph of the problem shown above. -> For that, we would first define the underlying graph of the problem as shown above.
the circuit below. -> Such a state, when the number of qubits is 4 (