Skip to content

Fix noise application order#202

Open
MoHermes wants to merge 6 commits into
devfrom
fix-noise-application-order
Open

Fix noise application order#202
MoHermes wants to merge 6 commits into
devfrom
fix-noise-application-order

Conversation

@MoHermes

@MoHermes MoHermes commented Jun 5, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions github-actions Bot added the bug Something isn't working label Jun 5, 2026
@MoHermes MoHermes requested review from Henri-ColibrITD, JulienCalistoTD and MathieuG-Colibri and removed request for JulienCalistoTD June 5, 2026 08:55

@MathieuG-Colibri MathieuG-Colibri left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Approved !

Comment on lines +340 to +344
qiskit_error_qubits = (
gate.controls + gate.targets
if isinstance(gate, ControlledGate)
else gate.targets
)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

gate.connections

Comment on lines +287 to +291
qiskit_error_qubits = (
gate.controls + gate.targets
if isinstance(gate, ControlledGate)
else gate.targets
)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

gate.connections

@Henri-ColibrITD Henri-ColibrITD left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

tiny tweaks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants