Skip to content

Fix citation and explanation in Floquet calibration example #4335

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

Merged

Conversation

balopat
Copy link
Contributor

@balopat balopat commented Jul 19, 2021

The Floquet calibration notebook's connection to our research needed more explanation. There is connection to both the quantum ring paper and the Fermi Hubbard paper. I also explained the motivation behind using segments on the line and added a quirk circuit to demonstrate the evolution of the charge density on a 5 qubit segment.

@balopat balopat requested review from cduck, vtomole, wcourtney and a team as code owners July 19, 2021 20:18
@balopat balopat requested a review from viathor July 19, 2021 20:18
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@balopat balopat changed the title Fix citation and explanation in floquet calibration example Fix citation and explanation in Floquet calibration example Jul 19, 2021
@google-cla google-cla bot added the cla: yes Makes googlebot stop complaining. label Jul 19, 2021
@balopat balopat requested review from mrwojtek and removed request for viathor July 19, 2021 20:19
@balopat
Copy link
Contributor Author

balopat commented Jul 19, 2021

cc @rmlarose

Copy link
Collaborator

@MichaelBroughton MichaelBroughton left a comment

Choose a reason for hiding this comment

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

LGTM. You could leave both references in , since they both talk about floquet calibration.

@rmlarose
Copy link
Contributor

IIRC we wanted to reference https://arxiv.org/pdf/2012.00921.pdf for that section because the benchmark circuit in the tutorial is the same as the one in the paper (see Fig. S3) except the tutorial has open boundary conditions and the paper has closed boundary conditions. It's less clear to me how the circuit in the tutorial corresponds to the Fermi-Hubbard experiment, but if I squint I can kind of see it. Either way if @mrwojtek prompted the change this LGTM.

I do like @MichaelBroughton's suggestion of referencing both papers, perhaps at the end of the tutorial ("For more on how Floquet calibration works, see...")

@mrwojtek
Copy link
Collaborator

The discussion with Balint started because of the error mitigation techniques that are present in the tutorial, they are closer to the ones in https://arxiv.org/pdf/2010.07965.pdf. However, the Hamiltonian there is more complicated and as you're saying Ryan you need to really squint to see it. The https://arxiv.org/pdf/2012.00921.pdf realizes exactly the Hamiltonian in this tutorial up to a boundary conditions (although in that paper the Hamiltonian is mentioned briefly in the text only).

I think what we really need is a sentence that the technique of averaging over many realizations is included in https://arxiv.org/pdf/2010.07965.pdf. The https://arxiv.org/pdf/2012.00921.pdf is better in terms of physics of this experiment.

@balopat
Copy link
Contributor Author

balopat commented Jul 20, 2021

Thanks for the comments. I am using this tutorial for a presentation and I wanted to understand the model we are using, and as it was, I found it confusing regarding the connection to the quantum ring paper: we had segments that were not connected by two-qubit gates, there were no single qubit gates between the sqrtISWAPs, I saw no mentioning of z densities. I reached out to @mrwojtek and he explained it from the Fermi-Hubbard (FH) point of view, which made more sense. But now I see the connection to the quantum ring (QR) paper too. And especially that both use Floquet calibration we could keep both in.

I'll rephrase:

  • the physics model could be looked at as a simplified version of either of the papers, with some explanation regarding the connection - FH with U=0, fixed theta to sqrtISWAP, single site excitation initial state, single chain; QR with open boundary condition, no complex hopping (hence no Z rotations between the sqrtISWAP gates)
  • it measures charge density which is closer to FH paper than the QR paper's Fourier transforms and spectroscopy direction
  • it uses post-selection that is in the FH paper and is not present in the QR paper
  • it runs the experiment segment_length/line_length times in parallel, and uses averages to smooth out heterogeneity on the chip - though the averaging technique was implemented in the FH paper across 16 different configurations, it was serial, not parallel

@balopat
Copy link
Contributor Author

balopat commented Jul 20, 2021

Please have a look again @rmlarose @mrwojtek @MichaelBroughton at the update. Thanks!

Copy link
Contributor

@rmlarose rmlarose left a comment

Choose a reason for hiding this comment

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

Thanks for the comments! Makes sense and the latest changes look great! I like in particular linking to the quirk circuit.

@balopat balopat added the automerge Tells CirqBot to sync and merge this PR. (If it's running.) label Jul 20, 2021
@CirqBot CirqBot added the front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. label Jul 20, 2021
@CirqBot CirqBot merged commit 093c171 into quantumlib:master Jul 20, 2021
@CirqBot CirqBot removed automerge Tells CirqBot to sync and merge this PR. (If it's running.) front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. labels Jul 20, 2021
@@ -212,7 +212,18 @@
"id": "a0Nrd1pkVWzo"
Copy link
Collaborator

Choose a reason for hiding this comment

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

single electron => single fermionic particle

quirk circuit is cool! Could you place the excitation in the middle (on the 2nd qubit, this is how it's being done in the tutorial).

Otherwise LGTM!


Reply via ReviewNB

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sounds good, I'll add those changes in!

rht pushed a commit to rht/Cirq that referenced this pull request May 1, 2023
…ib#4335)

The Floquet calibration notebook's connection to our research needed more explanation. There is connection to both the quantum ring paper and the Fermi Hubbard paper. I also explained the motivation behind using segments on the line and added a quirk circuit to demonstrate the evolution of the charge density on a 5 qubit segment.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Makes googlebot stop complaining.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants