Documentation for cirq.Circuit.prev_moment_operating_on is incorrect
#4894
Labels
kind/bug-report
Something doesn't seem to work.
Comments
CirqBot
pushed a commit
that referenced
this issue
Jan 25, 2022
See #4894 concerning a single-word documentation error for `cirq.Circuit.prev_moment_operating_on`
|
Closed by #4895. |
rht
pushed a commit
to rht/Cirq
that referenced
this issue
May 1, 2023
…mlib#4895) See quantumlib#4894 concerning a single-word documentation error for `cirq.Circuit.prev_moment_operating_on`
harry-phasecraft
pushed a commit
to PhaseCraft/Cirq
that referenced
this issue
Oct 31, 2024
…mlib#4895) See quantumlib#4894 concerning a single-word documentation error for `cirq.Circuit.prev_moment_operating_on`
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description of the issue
The docstring summaries for
cirq.Circuit.next_moment_operating_onandcirq.Circuit.prev_moment_operating_onare identical: both say "Finds the index of the next moment that touches the given qubits."The docstring for
cirq.Circuit.prev_moment_operating_onshould either say "Finds the index of the previous moment that touches the given qubits.", or else somehow specify that "next" means "next in a search backwards through the circuit".How to reproduce the issue
See links above, or examine docstring via
Cirq version
Issue identical in
cirqversions0.13.1and0.14.0.dev20220105215702The text was updated successfully, but these errors were encountered: