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

Remove qiskit.transpiler.synthesis for Qiskit 1.0 release #11570

Merged
merged 32 commits into from
Jan 25, 2024
Merged

Remove qiskit.transpiler.synthesis for Qiskit 1.0 release #11570

merged 32 commits into from
Jan 25, 2024

Conversation

ShellyGarion
Copy link
Member

Summary

Remove qiskit.transpiler.synthesis that has been deprecated in #11426

Details and comments

@ShellyGarion
deprecate transpiler/synthesis/graysynth.py
081c540
@ShellyGarion
style
de19803
@ShellyGarion
style
2bf99a3
@ShellyGarion
move aqc_plugin to qiskit/transpiler/passes/synthesis
f884e6d
@ShellyGarion
remove code from qiskit/transpiler/synthesis/aqc/aqc_plugin.py
54f0bab
@ShellyGarion
copy qiskit/transpiler/synthesis/aqc to qiskit/synthesis/unitary
2ea0016
@ShellyGarion
move tests from test/python/transpiler/aqc to test/python/synthesis/aqc
05b50e5
@ShellyGarion
update imports in aqc_plugin
3262d00
@ShellyGarion
add deprecation warning to AQC module
dc316df
@ShellyGarion
handle cyclic imports
4381198
@ShellyGarion
handle cyclic imports
6404bf9
@ShellyGarion
update link in docs
f88c2ff
@ShellyGarion
update init in qiskit/transpiler/synthesis/aqc
8078f2e
@ShellyGarion
style
bfaa805
@ShellyGarion
temporary remove deprecation warning test
cb042f3
@ShellyGarion
remove files from qiskit/transpiler/synthesis/aqc
3ebbc8c
@ShellyGarion
update link in test
aeb1a2d
@ShellyGarion
add release notes
e9fce2e
@ShellyGarion
update docs
cc35802
@ShellyGarion
update docs/apidocs/synthesis_aqc.rst
6b4cfec
@ShellyGarion
resolve conflict with main branch
cdb0d68
@ShellyGarion
add deprecations to qiskit/transpiler/synthesis/__init__.py
d1a5591
@ShellyGarion
Merge branch 'main' into transpiler_synth_deprecate
6ec572f
@ShellyGarion
Merge branch 'main' into transpiler_synth_deprecate
7e16f4a
@ShellyGarion
Merge branch 'main' into transpiler_synth_deprecate
947b01e
@ShellyGarion
fix link
52e851f
@ShellyGarion
remove qiskit/transpiler/synthesis
87e9035
@ShellyGarion
add release notes
65a6dde
@ShellyGarion ShellyGarion added on hold Can not fix yet Changelog: Removal Include in the Removed section of the changelog labels Jan 16, 2024
@ShellyGarion ShellyGarion added this to the 1.0.0 milestone Jan 16, 2024
@ShellyGarion ShellyGarion requested a review from Cryoris January 16, 2024 10:22
@ShellyGarion ShellyGarion requested review from alexanderivrii and a team as code owners January 16, 2024 10:22
@qiskit-bot
Copy link
Collaborator

One or more of the the following people are requested to review this:

@coveralls
Copy link

coveralls commented Jan 16, 2024 •
edited
Loading

Pull Request Test Coverage Report for Build 7652942663

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.01%) to 89.548%
Totals Coverage Status
Change from base Build 7645403491: 0.01%
Covered Lines: 59553
Relevant Lines: 66504
💛 - Coveralls

@Cryoris Cryoris changed the title Remove qiskit.transpler.synthesis for Qiskit 1.0 release Remove qiskit.transpiler.synthesis for Qiskit 1.0 release Jan 23, 2024
Cryoris
Cryoris approved these changes Jan 25, 2024
View reviewed changes
Copy link
Contributor

@Cryoris Cryoris left a comment

Choose a reason for hiding this comment

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

LGTM!

@Cryoris Cryoris removed the on hold Can not fix yet label Jan 25, 2024
@Cryoris Cryoris added this pull request to the merge queue Jan 25, 2024
Merged via the queue into Qiskit:main with commit 1c20b84 Jan 25, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Cryoris Cryoris Cryoris approved these changes

@alexanderivrii alexanderivrii Awaiting requested review from alexanderivrii alexanderivrii is a code owner

Assignees
No one assigned
Labels
Changelog: Removal Include in the Removed section of the changelog
Projects
None yet
Milestone
1.0.0
Development

Successfully merging this pull request may close these issues.
4 participants
@ShellyGarion @qiskit-bot @coveralls @Cryoris