Skip to content

💥 Remove obsolete QDMI calibration and pulse metadata - #2233

Draft
burgholzer wants to merge 1 commit into
mainfrom
codex/qdmi-v14-cleanup-adoption
Draft

💥 Remove obsolete QDMI calibration and pulse metadata#2233
burgholzer wants to merge 1 commit into
mainfrom
codex/qdmi-v14-cleanup-adoption

Conversation

@burgholzer

@burgholzer burgholzer commented Aug 24, 2026

Copy link
Copy Markdown
Member

🤖 AI text below 🤖

Description

Adopt QDMI's calibration-advisory and pulse-metadata removal independently of driver replacement, multi-program jobs, and program capabilities.

  • Remove C++ Device::getNeedsCalibration and Python Device.needs_calibration.
  • Remove the obsolete bundled-device property implementations and update generated bindings and documentation.
  • Preserve calibration submission, the CALIBRATION status, the existing format enum, and current compiler/concurrency behavior.
  • Document the device-property renumbering and required matching-header rebuild.

Dependencies and release scope

Based directly on main; no dependency on #2229, #2230, or #2231. During development, pin QDMI #513 at 52b0abb85beb022224f7441b0b92b5fb61dab91b, which depends only on QDMI #512. Target Core 4.1 / QDMI 1.4, never Core 4.0. Release artifacts must use a released QDMI version.

Validation

  • Full independent release build and CTest: 3,693 passed, one existing SC skip.
  • QDMI Python suite with both bundled devices enabled: 249 passed.
  • Stub generation, repository lint, C++ lint, and signed commit verification: passed.

Hosted CI remains the merge gate for the new head.

Checklist

  • The pull request only contains commits that are focused and relevant to this change.
  • I have added appropriate tests that cover the new/changed functionality.
  • I have updated the documentation to reflect these changes.
  • I have added entries to the changelog for any noteworthy additions, changes, fixes, or removals.
  • I have added migration instructions to the upgrade guide (if needed).
  • The changes follow the project's style guidelines and introduce no new warnings.
  • The changes are fully tested and pass the CI checks.
  • I have reviewed my own code changes.

If PR contains AI-assisted content:

  • Any agent that created, edited, or submitted GitHub content was explicitly authorized for that scope, as required by our AI Usage Guidelines.
  • Every agent-authored or agent-edited public text body begins with the visible disclosure 🤖 *AI text below* 🤖 (titles are exempt).
  • I have disclosed AI assistance in the PR description.
  • I confirm that I have personally reviewed and understood all AI-generated content, and accept full responsibility for it.

@burgholzer burgholzer added documentation Improvements or additions to documentation dependencies Pull requests that update a dependency file code quality Code quality improvements c++ Anything related to C++ code python Anything related to Python code QDMI Anything related to QDMI labels Aug 24, 2026
@burgholzer burgholzer self-assigned this Aug 24, 2026
@burgholzer
burgholzer force-pushed the codex/qdmi-v14-cleanup-adoption branch from afee5b2 to d83d5e1 Compare August 24, 2026 20:11
@codecov

codecov Bot commented Aug 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@burgholzer
burgholzer force-pushed the codex/qdmi-client-runtime-c1 branch from 3de68c6 to 32bb143 Compare August 24, 2026 23:47
@burgholzer
burgholzer force-pushed the codex/qdmi-v14-cleanup-adoption branch from d83d5e1 to 78c54a9 Compare August 24, 2026 23:47
@burgholzer burgholzer added this to the QDMI Support milestone Aug 26, 2026

@burgholzer burgholzer left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This one is actually straight forward and could be folded into #2226 to reduce the number of PRs in this stack a little and combine changes that logically belong to one another. After folding, this PR can be closed and the remaining PRs can be re-stacked accordingly.

@burgholzer
burgholzer force-pushed the codex/qdmi-client-runtime-c1 branch from 32bb143 to 7504600 Compare August 27, 2026 14:03
@burgholzer
burgholzer force-pushed the codex/qdmi-v14-cleanup-adoption branch from 78c54a9 to 3e01f3b Compare August 27, 2026 14:03
Remove calibration-advisory and pulse metadata independently of driver replacement and program capabilities. Preserve calibration submission and pin the isolated QDMI cleanup for Core 4.1 development.

Assisted-by: GPT-5.6 Sol via Codex
@burgholzer
burgholzer force-pushed the codex/qdmi-v14-cleanup-adoption branch from 3e01f3b to 70b11d6 Compare September 4, 2026 09:53
@burgholzer
burgholzer changed the base branch from codex/qdmi-client-runtime-c1 to main September 4, 2026 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++ Anything related to C++ code code quality Code quality improvements dependencies Pull requests that update a dependency file documentation Improvements or additions to documentation python Anything related to Python code QDMI Anything related to QDMI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant