Conversation
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
|
One or more of the following people are relevant to this code:
|
| <details> | ||
| <summary>**Observable estimation**</summary> | ||
|
|
||
| These tutorials focus on estimating physically meaningful quantities, such as energy or correlation values, by preparing quantum states and measuring observables. Techniques include both variational and Trotterized circuit approaches that balance circuit expressiveness with circuit-depth efficiency. Emphasis is placed on workflows that reduce quantum resource demands while maintaining accuracy, and enabling practical estimation of observables in chemical and physical systems. |
There was a problem hiding this comment.
Is this text intentionally removed? See also text removed from lines 66, 81, 135, 157, and 173.
| "</details>" | ||
| "<Accordion>\n", | ||
| " <AccordionItem title='Code for automated selection of \"zig-zag\" layout'>\n", | ||
| " <CodeCellPlaceholder tag=\"id-automated-layout-code\" />\n", |
There was a problem hiding this comment.
Not a blocker, but I'm surprised how much indentation (particularly on the right side!) of the code block happened when it was put inside the accordion. Do we have any control over how much margin is put around a code block inside the accordion? The image I attach here shows the code block in the accordion on the top, contrasted with a code block not in an accordion on the bottom.

| "<em>mathematical details</em>\n", | ||
| "\n", | ||
| "</summary>\n", | ||
| "<AccordionItem title=\"<em>mathematical details</em>\">\n", |
There was a problem hiding this comment.
| "<AccordionItem title=\"<em>mathematical details</em>\">\n", | |
| "<AccordionItem title=\"<em>Mathematical details</em>\">\n", |
pre-existing erroneous lower-case m
| "<Accordion>\n", | ||
| "\n", | ||
| "</summary>\n", | ||
| "<AccordionItem title=\"<em>mathematical details of hash collisions</em>\">\n", |
There was a problem hiding this comment.
| "<AccordionItem title=\"<em>mathematical details of hash collisions</em>\">\n", | |
| "<AccordionItem title=\"<em>Mathematical details of hash collisions</em>\">\n", |
beckykd
left a comment
There was a problem hiding this comment.
This is going to be gorgeous! Thanks Frank!!
| <summary> | ||
| Does Qiskit Runtime local testing mode support different execution modes? | ||
| </summary> | ||
| <span id="parallel"></span> |
There was a problem hiding this comment.
This ID belongs before the next accordion; not this one.
|
|
||
| </summary> | ||
| <Accordion> | ||
| <AccordionItem title="Optional reading — click on the triangle to read more about the experiment"> |
There was a problem hiding this comment.
| <AccordionItem title="Optional reading — click on the triangle to read more about the experiment"> | |
| <AccordionItem title="Optional reading — expand to read more about the experiment"> |
| "\n", | ||
| "</summary>\n", | ||
| "</AccordionItem>\n", | ||
| "<AccordionItem title=\"What are the principal advantages to SKQD over (a) SQD, and (b) KQD?\">\n", |
There was a problem hiding this comment.
| "<AccordionItem title=\"What are the principal advantages to SKQD over (a) SQD, and (b) KQD?\">\n", | |
| "<AccordionItem title=\"What are the principle advantages to SKQD over (a) SQD, and (b) KQD?\">\n", |
There was a problem hiding this comment.
Hm now that I see this, I think principal here is being used correctly, as an adjective - the most important advantages. Principle is only a noun, I believe
This PR switches some
<details>tags to the new<Accordion>component.I haven't switched all the tags; some will require rewording to make the titles fit on one line. I only switched tags that didn't require any rewording.