Skip to content
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

Fix UnassignedDurationError of update_from_instruction_schedule_map #10447

Merged

Conversation

to24toro
Copy link
Contributor

Summary

Update_from_instruction_schedule_map failed when the duration of schedules is treated as a parameter.
I added try.. except statement and set duration None when the duration is the parameter.

Details and comments

@to24toro to24toro requested a review from a team as a code owner July 19, 2023 08:16
@qiskit-bot
Copy link
Collaborator

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

  • @Qiskit/terra-core

@nkanazawa1989 nkanazawa1989 self-assigned this Jul 19, 2023
@nkanazawa1989 nkanazawa1989 added the Changelog: Bugfix Include in the "Fixed" section of the changelog label Jul 19, 2023
@nkanazawa1989
Copy link
Contributor

Thanks @to24toro could you please add a test to https://github.com/Qiskit/qiskit-terra/blob/a804df58c8e59c101fc3c13ef6a1a4cb172b9a5b/test/python/transpiler/test_target.py#L1144 and write a release note with bugfix?

@nkanazawa1989 nkanazawa1989 added the mod: transpiler Issues and PRs related to Transpiler label Jul 19, 2023
@coveralls
Copy link

Pull Request Test Coverage Report for Build 5597068363

  • 5 of 5 (100.0%) changed or added relevant lines in 1 file are covered.
  • 44 unchanged lines in 6 files lost coverage.
  • Overall coverage increased (+0.02%) to 86.071%

Files with Coverage Reduction New Missed Lines %
qiskit/transpiler/passes/layout/dense_layout.py 1 95.1%
crates/qasm2/src/lex.rs 2 91.14%
qiskit/extensions/quantum_initializer/squ.py 2 80.0%
qiskit/transpiler/preset_passmanagers/level1.py 3 96.67%
qiskit/transpiler/preset_passmanagers/common.py 10 91.58%
qiskit/dagcircuit/dagcircuit.py 26 90.27%
Totals Coverage Status
Change from base Build 5594502856: 0.02%
Covered Lines: 72688
Relevant Lines: 84451

💛 - Coveralls

Copy link
Contributor

@nkanazawa1989 nkanazawa1989 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @to24toro this looks good to me.

@nkanazawa1989 nkanazawa1989 added this pull request to the merge queue Jul 20, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 20, 2023
@jakelishman jakelishman added this pull request to the merge queue Jul 20, 2023
Merged via the queue into Qiskit:main with commit ac29776 Jul 20, 2023
13 checks passed
to24toro added a commit to to24toro/qiskit-terra that referenced this pull request Aug 3, 2023
…iskit#10447)

* fix UnassignedDurationError of update_from_instruction_schedule_map

* add test

* add reno

* rename test

* fix reno

* Update releasenotes/notes/fix-update-from-instruction-schedule-map-d1cba4e4db4b679e.yaml

---------

Co-authored-by: Naoki Kanazawa <nkanazawa1989@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changelog: Bugfix Include in the "Fixed" section of the changelog mod: transpiler Issues and PRs related to Transpiler
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants