refactor: remove all deprecated code #1248
Conversation
This commit removes all deprecated code. Even though this is done a bit earlier than originally announced under the Qiskit deprecation policy, we are taking this step now since this project has become part of a community effort. As a direct consequence, new codeowners will soon join the team behind Qiskit Nature. Rather than handing over a codebase that includes pieces of code which need attention in the near future (and even continued support until then) we are removing these parts early. As a consequence, if you still require parts of this code, we suggest you stick to Qiskit Nature 0.6 until you can transition based on the provided migration guides. This commit also removes the deprecation utilities which are superseded by the ones provided by Qiskit directly (see qiskit.utils.deprecation).
Now that the old code covered by the migration guide is being removed, doctest can no longer execute this code. Thus, this commit replaces the doctest directives with simple code cells.
mrossinek
requested review from
stefan-woerner,
woodsp-ibm and
ftroisi
as code owners
|
This also closes #1169 |
|
Since there is too much to go through in detail in this PR, here are a few noteworthy details:
|
|
What do you want to do about the AdaptVQE howto |
Nice catch! I did not think to look through the tutorials and how-tos for additional usage of settings or other removed aspects. Will do that now 👍 |
|
I have pushed two additional changes:
|
* refactor: remove all deprecated code This commit removes all deprecated code. Even though this is done a bit earlier than originally announced under the Qiskit deprecation policy, we are taking this step now since this project has become part of a community effort. As a direct consequence, new codeowners will soon join the team behind Qiskit Nature. Rather than handing over a codebase that includes pieces of code which need attention in the near future (and even continued support until then) we are removing these parts early. As a consequence, if you still require parts of this code, we suggest you stick to Qiskit Nature 0.6 until you can transition based on the provided migration guides. This commit also removes the deprecation utilities which are superseded by the ones provided by Qiskit directly (see qiskit.utils.deprecation). * docs: remove doctest from migration guides Now that the old code covered by the migration guide is being removed, doctest can no longer execute this code. Thus, this commit replaces the doctest directives with simple code cells. * docs: add 0.7 prelude * docs: add spelling exceptions * docs: fix indent in release note * fix: remove final occurrence of opflow * lint: run black * fix: remove final occurrences of removed setting attributes * docs: removes a now obsolete release note * fix: remove left over setting
Summary
This commit removes all deprecated code. Even though this is done a bit earlier than originally announced under the Qiskit deprecation policy, we are taking this step now since this project has become part of a community effort.
As a direct consequence, new codeowners will soon join the team behind Qiskit Nature. Rather than handing over a codebase that includes pieces of code which need attention in the near future (and even continued support until then) we are removing these parts early.
As a consequence, if you still require parts of this code, we suggest you stick to Qiskit Nature 0.6 until you can transition based on the provided migration guides.
This commit also removes the deprecation utilities which are superseded by the ones provided by Qiskit directly (see qiskit.utils.deprecation). Thus, this PR closes #1196
Details and comments