-
Notifications
You must be signed in to change notification settings - Fork 17
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
Unittest not running because QiskitTestCase
class is removed in Qiskit 1.0.
#98
Comments
QiskitTestCase
class is not defined in Qiskit 1.0.QiskitTestCase
class is removed in Qiskit 1.0.
Indeed, |
Hello @1ucian0 okay! I will try this and open a PR when done. |
Hello, @1ucian0 is there any documentation explaining how to substitute iteration of CircuitInstruction in Qiskit 2.0 using the update:I was able to find the documentation and I think I resolved the problem. |
Great! want to push somewhere? I could have a look. |
Hello, I only fixed that particular issue, I still have some others to solve (I will also need to review the formatting before opening a PR). They will all be on this branch https://github.com/robotAstray/qiskit-cold-atom/tree/fix-qiskit-terra-dependencies |
@1ucian0 Since we need to move from
What would be the requirements for (1) and the instructions for (2)? something like the code used elsewhere?
update: these methods should also be implemented in |
Information
What is the current behavior?
As per issue #97, the
qiskit-terra (>=0.17.1)
dependencies need to be removed. I have also noticed that the unittest files depend on theqiskit.test
classQiskitTestCase
, which is not available in Qiskit 1.0 and beyond.Steps to reproduce the problem
qiskit_are
,qiskit-nature
,qiskit-algorithms
)qiskit_cold_atom/applications/time_evolution_solver.py
to:(this change will ensure the use of the
qiskit-algorithm
rather than the deprecatedqiskit.algorithm.
python -m unittest
in the root folderThis is because
qiskit.test
classQiskitTestCase
is not defined in Qiskit 1.0What is the expected behavior?
Unittests should run and either pass or fail.
Suggested solutions
[update at on 3/09/24 at 16:43GMT on possible solution]
qiskit.test
from qiskit-cold-atom and use an alternative methodI think this problem should be addressed before (or while) resolving issue #97
The text was updated successfully, but these errors were encountered: