Skip to content

Fix docstring for cirq.map_operations#5849

Merged
tanujkhattar merged 14 commits intomasterfrom
docstring_fix
Aug 31, 2022
Merged

Fix docstring for cirq.map_operations#5849
tanujkhattar merged 14 commits intomasterfrom
docstring_fix

Conversation

@tanujkhattar
Copy link
Copy Markdown
Collaborator

Highlights that map_func takes a moment index along with the original operation.

@tanujkhattar tanujkhattar requested review from a team, cduck and vtomole as code owners August 31, 2022 21:44
@tanujkhattar tanujkhattar requested a review from dabacon August 31, 2022 21:44
@CirqBot CirqBot added the size: S 10< lines changed <50 label Aug 31, 2022
Copy link
Copy Markdown
Collaborator

@pavoljuhas pavoljuhas left a comment

Choose a reason for hiding this comment

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

LGTM

@tanujkhattar tanujkhattar merged commit e5a11b6 into master Aug 31, 2022
rht pushed a commit to rht/Cirq that referenced this pull request May 1, 2023
* Fix docstrings for cirq.map_operations
harry-phasecraft pushed a commit to PhaseCraft/Cirq that referenced this pull request Oct 31, 2024
* Fix docstrings for cirq.map_operations
@pavoljuhas pavoljuhas deleted the docstring_fix branch January 22, 2025 07:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: S 10< lines changed <50

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants