Treat Reset like Measure in ConstrainedReschedule

[hunterkemeny]

Summary

fixes #10354 ConstrainedReschedule raises an error with reset

Details and comments

Reset and Measure (mid-circuit) both provide duration data from the backend, but this is untrustworthy. To reflect this, I added RuntimeWarnings in qiskit/transpiler/passes/scheduling/base_scheduler.py.

That Reset reports duration data is not recognized in ConstrainedReschedule, leading to the error in Issue #10354. Thus, I added the same handling for Reset in ConstrainedReschedule as is used for Measure.

[qiskit-bot]

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

• @Qiskit/terra-core
• @nkanazawa1989

[coveralls]

Pull Request Test Coverage Report for Build 8070375077

Warning: 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.

• For more information on this, see Tracking coverage changes with pull request builds.
• To avoid this issue with future PRs, see these Recommended CI Configurations.
• For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

• -4 of 13 (69.23%) changed or added relevant lines in 2 files are covered.
• 466 unchanged lines in 57 files lost coverage.
• Overall coverage increased (+0.1%) to 89.291%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
qiskit/transpiler/passes/scheduling/base_scheduler.py	8	9	88.89%
qiskit/transpiler/passes/scheduling/alignments/reschedule.py	1	4	25.0%

Files with Coverage Reduction	New Missed Lines	%
crates/qasm2/src/expr.rs	1	93.81%
qiskit/circuit/controlflow/control_flow.py	1	95.0%
qiskit/circuit/library/standard_gates/global_phase.py	1	95.83%
qiskit/circuit/library/standard_gates/rx.py	1	98.55%
qiskit/circuit/library/standard_gates/ry.py	1	98.51%
qiskit/circuit/library/standard_gates/ryy.py	1	97.3%
qiskit/circuit/library/standard_gates/rz.py	1	98.46%
qiskit/circuit/library/standard_gates/rzz.py	1	97.06%
qiskit/circuit/library/standard_gates/swap.py	1	98.08%
qiskit/circuit/library/standard_gates/u3.py	1	98.9%

Totals
Change from base Build 7830061937:	0.1%
Covered Lines:	59130
Relevant Lines:	66222

---

💛 - Coveralls

[nkanazawa1989]

Thanks @hunterkemeny for adding support for reset instruction. Since Qiskit is assumed to be vendor agnostic (I'm not sure if this can be true for low level operations like a scheduling), the warning message should be more generic. I added some suggestions but feel free to update if you have better wording.

Let me confirm your intention. As you may notice ConstrainedReschedule is not a BaseScheduler subclass where you added the user warnings. This means user will never see the message when they only run the rescheduler. In principle rescheduler doesn't work without scheduling, so you are expecting the user will see the message at first scheduling step, then no additional error will raise in rescheduler, right?

[nkanazawa1989]

Thanks, this looks good to me.