Skip to content

Conversation

@SamFerracin
Copy link
Collaborator

This PR adds docs for the Executor class

Copy link
Collaborator

@TsafrirA TsafrirA left a comment

Choose a reason for hiding this comment

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

Some minor comments to kick things off.


* a :class:`~qiskit.circuit.QuantumCircuit` with static, non-parametrized gates;
* or a parametrized :class:`~qiskit.circuit.QuantumCircuit`, together with an array of parameter values;
* or a parametrized :class:`~qiskit.circuit.QuantumCircuit`, together with a
Copy link
Collaborator

Choose a reason for hiding this comment

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

Input parameters might also be needed here.

Copy link
Collaborator

Choose a reason for hiding this comment

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

i'm on the fence, but i think it's okay to omit the existence of samplex arguments to keep the bullets short

SamFerracin and others added 5 commits November 6, 2025 11:19
:class:`~samplomatic.samplex.Samplex`, which is responsible for generating randomized sets of
parameters for the given circuit. As part of the :class:`~samplomatic.samplex.Samplex` arguments,
we provide ``10`` sets of parameters for the parametric gates in the original circuit.
Additionally, we use the ``shape`` request argument to request an extension of the implicit shape
Copy link
Collaborator

Choose a reason for hiding this comment

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

IF we have a broadcasting section, we should link to it here

SamFerracin and others added 8 commits November 6, 2025 14:43
Co-authored-by: Ian Hincks <ian.hincks@gmail.com>
Co-authored-by: Ian Hincks <ian.hincks@gmail.com>
Co-authored-by: Ian Hincks <ian.hincks@gmail.com>
Co-authored-by: Ian Hincks <ian.hincks@gmail.com>
Co-authored-by: Ian Hincks <ian.hincks@gmail.com>
Co-authored-by: Ian Hincks <ian.hincks@gmail.com>
Co-authored-by: Ian Hincks <ian.hincks@gmail.com>
@ihincks ihincks merged commit 5dd13ed into Qiskit:executor_preview Nov 6, 2025
19 checks passed
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.

3 participants