Remove schedule, sequence and related unit tests#13809
Conversation
ElePT
requested review from
a team,
eggerdj,
nkanazawa1989 and
wshanks
as code owners
|
One or more of the following people are relevant to this code:
|
| qubits = [qubit_map[q] for q in node.qargs] | ||
| _pass.get_calibration(node.op, qubits) | ||
|
|
||
| def test_ecr_cx_forward(self): |
There was a problem hiding this comment.
These tests could arguably be kept and be rewritten to assert based on the the pulse gates more directly. But since the pass deprecation is in the merge queue currently I think it's fine to just drop this now.
There was a problem hiding this comment.
This test has actually just been removed in main, so problem solved :)
| The functions ``sequence`` and ``schedule`` from the :mod:`.compiler` | ||
| module have been removed following their deprecation in Qiskit 1.3. | ||
| These functions depend on and relate to the Pulse package which is | ||
| also being removed in Qiskit 2.0. No newline at end of file |
There was a problem hiding this comment.
I might expand on this a bit more to explain there isn't an alternative to this because it depends on being able to translate circuits to pulse from backend definitions that is not present. But we can do this in the release round-up PR.
There was a problem hiding this comment.
Expanded in c6f1366, but we should probably improve it in the release PR (as most of the removal renos that relate to these large removal epics).
…ecated and the whole docstring will go soon)
Pull Request Test Coverage Report for Build 13203145301Warning: This coverage report may be inaccurate.This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Details
💛 - Coveralls |
4 participants
Summary
This PR removes the
scheduleandsequencefunctions deprecated in qiskit 1.3 as part of the Pulse deprecation effort.Details and comments
Part of #13662