Skip to content

Deprecate classes and paramters in contrib#5684

Merged
dstrain115 merged 6 commits intoquantumlib:masterfrom
dstrain115:deprecate_contrib
Jul 8, 2022
Merged

Deprecate classes and paramters in contrib#5684
dstrain115 merged 6 commits intoquantumlib:masterfrom
dstrain115:deprecate_contrib

Conversation

@dstrain115
Copy link
Copy Markdown
Collaborator

- Deprecates parameters in contrib
- Removes MergeNQubits and StrategyExecutor in favor of transformers

@dstrain115 dstrain115 requested review from a team, cduck and vtomole as code owners July 8, 2022 15:55
@dstrain115 dstrain115 requested a review from mpharrigan July 8, 2022 15:55
@CirqBot CirqBot added the size: M 50< lines changed <250 label Jul 8, 2022
# See the License for the specific language governing permissions and
# limitations under the License.

from typing import Iterator
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

odd that you have to add this!

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Yeah, not sure why it was missing before, but I figured we should add it while I was here.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

the copyright or the import? The import was indeed there

@dstrain115 dstrain115 merged commit 7c9c779 into quantumlib:master Jul 8, 2022
rht pushed a commit to rht/Cirq that referenced this pull request May 1, 2023
* Deprecate classes and paramters in contrib

- Deprecates parameters in contrib
- Removes MergeNQubits and StrategyExecutor in favor of transformers
harry-phasecraft pushed a commit to PhaseCraft/Cirq that referenced this pull request Oct 31, 2024
* Deprecate classes and paramters in contrib

- Deprecates parameters in contrib
- Removes MergeNQubits and StrategyExecutor in favor of transformers
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: M 50< lines changed <250

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants