Closed dwierichs closed 1 week ago
Hello. You may have forgotten to update the changelog!
Please edit doc/releases/changelog-dev.md
with:
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 99.66%. Comparing base (
14a0b63
) to head (f28f967
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Thanks @mudit2812 for the detailed feedback! :rocket: Regarding the structure, I went for your suggestion, except for the last part (configuring MCMs) being part of the "Simulating MCMs" section.
Regarding applying transforms vs QNode config, I would suggest that for now we do not explicitly discourage users from using the transforms, but we do so implicitly by advertising the config path in the examples.
Couple of weird things are happening:
The panel on the left side is showing a "Simulation of MCMs" page, but it just links to the "Simulation of MCMs" section inside the MCMs doc.
The contents table for the MCMs doc (right side of the page) doesn't show the "Simulation of MCMs" section.
Context: The PennyLane documentation has a quickstart page on measurements, which includes a large section about mid-circuit measurements. With the focus on dynamic circuits and mid-circuit measurements more generally, we would like to make the content on MCMs more visible.
Description of the Change: Extracts the MCM part of the measurements quickstart and adds it as its own quickstart page. Adds some details on simulation techniques and polishes the content.
Benefits: Visibility, documentation extent for MCMs
Possible Drawbacks:
Related GitHub Issues:
[sc-65318]