Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Change input to SimulatesAmplitudes.compute_amplitudes from 2-d numpy array to 1-d int sequence #2151

Closed
kevinsung opened this issue Sep 23, 2019 · 2 comments · Fixed by #2158
Closed

Change input to SimulatesAmplitudes.compute_amplitudes from 2-d numpy array to 1-d int sequence #2151

kevinsung opened this issue Sep 23, 2019 · 2 comments · Fixed by #2158
Assignees
@kevinsung

Comments

@kevinsung
Copy link
Collaborator

@viathor are you doing this in #2114 ? If not I can do it.
@viathor
Copy link
Collaborator

viathor commented Sep 23, 2019

I'd rather keep the PR focused on one change, so nope. Feel free to do this. If you don't have time, I can also take it. Your call.

@kevinsung kevinsung self-assigned this Sep 23, 2019
@kevinsung
Copy link
Collaborator Author

Ok I'll do it.

@viathor viathor mentioned this issue Sep 23, 2019
Merged
@kevinsung kevinsung mentioned this issue Sep 24, 2019
Merged
CirqBot pushed a commit that referenced this issue Sep 24, 2019
@kevinsung @CirqBot
Change compute_amplitudes input format from 2-d array of bools to 1-d…
91a0dd1 
… array of ints (#2158)

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

@kevinsung kevinsung

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Change compute_amplitudes input format from 2-d array of bools to 1-d array of ints kevinsung/Cirq
2 participants
@kevinsung @viathor