Skip to content

Conversation

kt474
Copy link
Member

@kt474 kt474 commented Apr 25, 2025

Summary

Can't merge until 3 months after 0.38.0 release ~ 7/16

Details and comments

Fixes #2179

@kt474 kt474 added the on hold label Apr 25, 2025
@kt474 kt474 marked this pull request as ready for review May 30, 2025 16:35
@kt474 kt474 added the Changelog: Removal Include in the Removed section of the changelog label Jun 16, 2025
@kt474 kt474 changed the title WIP Completely remove pulse defaults Completely remove pulse defaults Jun 26, 2025
@kt474 kt474 removed the on hold label Jul 15, 2025
@kt474 kt474 requested a review from ElePT July 15, 2025 17:51
@kt474
Copy link
Member Author

kt474 commented Jul 15, 2025

@ElePT This can be part of the 0.41.0 release too.

@ElePT ElePT added this to the 0.41.0 milestone Jul 16, 2025
Copy link
Collaborator

@ElePT ElePT left a comment

Choose a reason for hiding this comment

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

Sorry, I forgot to submit the review yesterday. I think that the changes look good, I just had a couple of minor comments.

Copy link
Collaborator

@ElePT ElePT left a comment

Choose a reason for hiding this comment

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

LGTM! Happy to see technical debt go :)

@ElePT ElePT added this pull request to the merge queue Jul 22, 2025
Merged via the queue into Qiskit:main with commit 00e67a4 Jul 22, 2025
20 checks passed
ihincks pushed a commit to ihincks/qiskit-ibm-runtime that referenced this pull request Sep 3, 2025
* wip remove pulse defaults

* remove fake backend defaults

* fix docs build

* fix docs again

* unit tests

* cleanup unused methods

* lint and reno

* add warning back

* Update release-notes/unreleased/2238.other.rst

Co-authored-by: Elena Peña Tapia <57907331+ElePT@users.noreply.github.com>

---------

Co-authored-by: Elena Peña Tapia <57907331+ElePT@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changelog: Removal Include in the Removed section of the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cleanup all pulse related code
2 participants