Overhaul Qmod Tutorial (Part I & II) - #1714
Conversation
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Rewrite Qmod_tutorial_part1 and Qmod_tutorial_part2 into a cohesive two-part Qmod tutorial: a run-and-see warmup, numbered concept/exercise sections, and a collected Solutions section per part. Both notebooks execute end-to-end and pass the testbook harness under the CI config (filterwarnings=error). Fixes in Part II: removed a stray %%sql cell and renamed a Hamiltonian variable that had shadowed the built-in Hadamard gate H. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
7a6ddaa to
e5f570c
Compare
| @@ -5,518 +5,376 @@ | |||
| "id": "0", | |||
There was a problem hiding this comment.
They generate a flat unrolled description, which can impact compilation performance.
I don't think that this statement is useful here. This is a beginner tutorial, and this information isn't beginner-friendly. I would introduce it later, perhaps in the advanced tutorial or the user guide itself.
Reply via ReviewNB
There was a problem hiding this comment.
Are you talking about the note as a whole, or just the second sentence? I need the note as a whole because QAEs are teaching people, in a first Qmod training, to use Python control flow by default, and I need to mention this as a possibility, but advise against it for the reasons I mention. Indeed this should be covered in more depth in an advanced tutorial.
There was a problem hiding this comment.
I'm talking about specifically "They generate a flat unrolled description" - if the targeted reader doesn't yet know the terminology this will sound like a lot of words that they can't comprehend
| @@ -4,366 +4,320 @@ | |||
| "cell_type": "markdown", | |||
There was a problem hiding this comment.
This cell is not looking well, but might be just a rendering problem from reviewNB
Reply via ReviewNB
There was a problem hiding this comment.
What are you seeing? What specifically is messed up?
There was a problem hiding this comment.
Font is bigger and bold. + the paragraph starts with a >
There was a problem hiding this comment.
Now it renders fine, but it was looking like a header. It was probably some reviewNB bug
| @@ -4,366 +4,320 @@ | |||
| "cell_type": "markdown", | |||
There was a problem hiding this comment.
Suggestion: Exercise C: Measure <Z_i> for a system with N>i+1 qubits. This is important because Z_i is a Pauli operator acting over i+1 qubits, while you need to measure an N-qubit system.
Reply via ReviewNB
There was a problem hiding this comment.
I actually consider this a bug. I filed a ticket and synced Dror.
| @@ -4,366 +4,320 @@ | |||
| "cell_type": "markdown", | |||
There was a problem hiding this comment.
There was a problem hiding this comment.
It renders fine. I think it was reviewNB.
3 participants
Summary
Rewrites the two Classiq Tutorial notebooks (
Qmod_tutorial_part1.ipynb,Qmod_tutorial_part2.ipynb) into a cohesive two-part Qmod Tutorial:suzuki_trotter/observe, higher-order functions (incl. built-inqpe), execution parameters & hybrid execution.# TODOexercise skeleton, with a collected Solutions section per notebook (every solution is a complete, executable cell, so CI actually runs it).🤖 Generated with Claude Code