Remove deprecated code #690
Conversation
Pull Request Test Coverage Report for Build 6224938528
💛 - Coveralls |
adekusar-drl
requested review from
stefan-woerner and
woodsp-ibm
as code owners
There was a problem hiding this comment.
It all looks fine to me. Since its closing the issue, and since it dealt with all the task items listed there I checked them all off in the issue.
One comment I will make, since it had to be changed (well only the format not the content) is the 0..5 migration guide. The classes, like the older QNNs and QuantumKernel often are linked, and these presently do work, but will no longer do so once the new docs, with this guide, are republished. Same is true of older release notes too of course where the same may happen. In the case of the guide there are lot of links though.
|
The problem with links is that we don't keep documentation for all versions. If we could have all releases, then we would be able to keep them correct. |
|
Though in this case they are all absolute links so it would not help - in principle yes! |
|
Surely, we would have to include something like |
|
Unless they were all relative urls so once you navigated to whatever version folder they were copied to, once they were no longer the current version, they would all still work. Anyway something for the future if/when versions of docs ever happen |
* remove old qnns * remove old kernel * remove warning filtration * fix tests * fix style * replace migration notebook with rst * update reno * skip statevector test on macos * remove sampling neural network * remove deprecations * new tests * new tests * fix stestr * replace bind_parameters with assign_parameters * remove stestr fix
Summary
This PR removes:
CircuitQNNEstimatorQNNTwoLayerQNNQuantumKernelFixes #686