💥 Remove the calibration advisory property - #512
Open
burgholzer wants to merge 1 commit into
Open
Conversation
burgholzer
added
documentation
Contributor
|
burgholzer
force-pushed
the
codex/remove-needs-calibration
branch
2 times, most recently
from
August 24, 2026 19:30
2583d9b to
d358e6b
Compare
11 tasks
burgholzer
force-pushed
the
codex/remove-needs-calibration
branch
from
August 24, 2026 19:35
d358e6b to
2c636b9
Compare
burgholzer
marked this pull request as ready for review
August 24, 2026 19:42
burgholzer
force-pushed
the
codex/remove-needs-calibration
branch
from
September 4, 2026 09:28
2c636b9 to
f921642
Compare
burgholzer
changed the base branch from
codex/client-driver-abi-stable-ids
to
develop
September 4, 2026 09:33
This was referenced Sep 4, 2026
Drop the unused NEEDSCALIBRATION device property and close the resulting enum gap. Keep calibration-job submission and the distinct calibration status. Keep this cleanup independent of payload and driver changes. Mark its breaking headers as 1.4.0-dev instead of inheriting the 1.3.4 development version. Assisted-by: GPT-5.6 Sol via Codex
burgholzer
force-pushed
the
codex/remove-needs-calibration
branch
from
September 5, 2026 14:29
f921642 to
3e2e11a
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🤖 AI text below 🤖
Description
Remove the unused
QDMI_DEVICE_PROPERTY_NEEDSCALIBRATIONadvisory and close the regular property-enum gap. PreserveQDMI_DEVICE_STATUS_CALIBRATIONand calibration-job submission. Provider-specific calibration readiness can use a custom property.This is the first independent metadata-cleanup layer, based directly on
v1.4; #513 follows it. It does not include payload capabilities, multi-program jobs, or replaceable drivers. The breaking development headers are identified as 1.4.0-dev.Validation
uvx prek run -a: passed.AI assistance: Codex separated the stack, adapted the cleanup to develop, and ran local validation.
Consumer compatibility
Removing the advisory does not remove calibration-job submission or the CALIBRATION status. Keep provider-specific calibration readiness and workflows in documented extensions; no common calibration workflow is standardized here.
Before the interface change merges, link a working Core consumer and a demonstration with at least one existing provider, preferably both where applicable. Compatibility evidence suffices when a provider needs no changes. Development revisions may support these tests; published artifacts must use released dependencies. Core adoption: #2233.
Checklist
If PR contains AI-assisted content:
Assisted-by: [Model Name] via [Tool Name]footer.