Skip to content

Use more accurate complex128 for testing non-public method #5616

Merged
CirqBot merged 2 commits intoquantumlib:masterfrom
dabacon:flakediag
Jun 25, 2022
Merged

Use more accurate complex128 for testing non-public method #5616
CirqBot merged 2 commits intoquantumlib:masterfrom
dabacon:flakediag

Conversation

@dabacon
Copy link
Copy Markdown
Collaborator

@dabacon dabacon commented Jun 24, 2022

And loosen the atol for the test.

@dabacon dabacon requested review from a team, cduck and vtomole as code owners June 24, 2022 22:51
@dabacon dabacon requested a review from 95-martin-orion June 24, 2022 22:51
@CirqBot CirqBot added the Size: XS <10 lines changed label Jun 24, 2022
@MichaelBroughton MichaelBroughton added the automerge Tells CirqBot to sync and merge this PR. (If it's running.) label Jun 24, 2022
@MichaelBroughton MichaelBroughton self-assigned this Jun 24, 2022
@CirqBot CirqBot added the front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. label Jun 25, 2022
@CirqBot CirqBot merged commit f370010 into quantumlib:master Jun 25, 2022
@CirqBot CirqBot removed automerge Tells CirqBot to sync and merge this PR. (If it's running.) front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. labels Jun 25, 2022
CirqBot pushed a commit that referenced this pull request Jun 29, 2022
…final_state_vector (#5636)

Rolls back change in #5426 that set the default precision on `Circuit#unitary` to `np.complex128`.

Also sets the default on `Circuit#final_state_vector` to `np.complex128`.  There was a deprecation warning that this was going to be downgraded to `np.complex64`,, but this will now remain `np.complex128`.  This is breaking in that is someone explicitly passed `dtype=None` to this method it would now break.  

Rolls back #5426 #5616 #5537 #5535
rht pushed a commit to rht/Cirq that referenced this pull request May 1, 2023
rht pushed a commit to rht/Cirq that referenced this pull request May 1, 2023
…final_state_vector (quantumlib#5636)

Rolls back change in quantumlib#5426 that set the default precision on `Circuit#unitary` to `np.complex128`.

Also sets the default on `Circuit#final_state_vector` to `np.complex128`.  There was a deprecation warning that this was going to be downgraded to `np.complex64`,, but this will now remain `np.complex128`.  This is breaking in that is someone explicitly passed `dtype=None` to this method it would now break.  

Rolls back quantumlib#5426 quantumlib#5616 quantumlib#5537 quantumlib#5535
harry-phasecraft pushed a commit to PhaseCraft/Cirq that referenced this pull request Oct 31, 2024
harry-phasecraft pushed a commit to PhaseCraft/Cirq that referenced this pull request Oct 31, 2024
…final_state_vector (quantumlib#5636)

Rolls back change in quantumlib#5426 that set the default precision on `Circuit#unitary` to `np.complex128`.

Also sets the default on `Circuit#final_state_vector` to `np.complex128`.  There was a deprecation warning that this was going to be downgraded to `np.complex64`,, but this will now remain `np.complex128`.  This is breaking in that is someone explicitly passed `dtype=None` to this method it would now break.  

Rolls back quantumlib#5426 quantumlib#5616 quantumlib#5537 quantumlib#5535
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Size: XS <10 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants