Skip to content

CLS-7784: sawtooth notebook — bind num_kicks for analysis cell - #1582

Draft
din-carmon wants to merge 1 commit into
Classiq:mainfrom
din-carmon:core/din_carmon/CLS-7784-sawtooth-use-assign-parameters
Draft

CLS-7784: sawtooth notebook — bind num_kicks for analysis cell#1582
din-carmon wants to merge 1 commit into
Classiq:mainfrom
din-carmon:core/din_carmon/CLS-7784-sawtooth-use-assign-parameters

Conversation

@din-carmon

@din-carmon din-carmon commented May 25, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

The depth/cx-count analysis cell now calls assign_parameters once to bind
num_kicks=1, then reads transpiled_circuit.depth / count_ops on the bound
qprog. The original parametric qprog is still used for the simulator and
hardware execution cells, which bind num_kicks per call via batch_sample.

This pattern works against both today's production (old flow, after the
Cadmium PR that teaches assign_parameters' QASM-only path about
synthesis-time structural params) and the future new-flow rollout (where
assign_parameters has always routed structural params through the qmod
path).
@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@din-carmon
din-carmon marked this pull request as draft May 25, 2026 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant