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.

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

Deprecating cross_entropy_benchmarking and grid_parallel_two_qubit_xeb #5424

Merged
merged 8 commits into from
May 31, 2022

Conversation

dstrain115
Copy link
Collaborator

  • cirq.experiments.cross_entropy_benchmarking is redundant and not easy
    to adapt.
  • cirq.experiments.grid_parallel_two_qubit_xeb moved to
    recirq.benchmarks.xeb
  • Also delete cross_entropy_benchmarking example

This is part of #3775.

- cirq.experiments.cross_entropy_benchmarking is redundant and not easy
  to adapt.
- cirq.experiments.grid_parallel_two_qubit_xeb moved to
  recirq.benchmarks.xeb
- Also delete cross_entropy_benchmarking example

This is part of quantumlib#3775.
@dstrain115 dstrain115 requested review from mrwojtek, a team, vtomole and cduck as code owners May 31, 2022 19:56
@CirqBot CirqBot added the size: L 250< lines changed <1000 label May 31, 2022
Copy link
Collaborator

@MichaelBroughton MichaelBroughton left a comment

Choose a reason for hiding this comment

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

LGTM, minor nit might be to break down some of the higher count assert_deprecateds just to make sure we aren't double warning in some places and not warning at all in others.

# Sanity test that plot runs.
ax = plt.subplot()
results_1.plot(ax)
with cirq.testing.assert_deprecated(_DEPRECATION_MESSAGE, deadline='v0.16', count=10):
Copy link
Collaborator

Choose a reason for hiding this comment

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

Could we use a few more contexts with lower counts that are a bit more tightly coupled to the things that are deprecated ? Just want to make sure we aren't accidentally raising two deprecations on some fxn calls.

@dstrain115 dstrain115 added the automerge Tells CirqBot to sync and merge this PR. (If it's running.) label May 31, 2022
@CirqBot CirqBot added the front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. label May 31, 2022
@CirqBot CirqBot merged commit 5ad8e6e into quantumlib:master May 31, 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 May 31, 2022
rht pushed a commit to rht/Cirq that referenced this pull request May 1, 2023
quantumlib#5424)

- cirq.experiments.cross_entropy_benchmarking is redundant and not easy
  to adapt.
- cirq.experiments.grid_parallel_two_qubit_xeb moved to
  recirq.benchmarks.xeb
- Also delete cross_entropy_benchmarking example

This is part of quantumlib#3775.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size: L 250< lines changed <1000
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants