🐛 Initialize the Qiskit C API once - #2223
Merged
Merged
Conversation
Publish the vendored translation-unit-local function tables through thread-safe static initialization for nanobind 3 free-threaded bindings. Add concurrent import and export coverage. Assisted-by: GPT-5.6 Sol via Codex Signed-off-by: Simon Hofmann <simon.t.hofmann@tum.de>
simon1hofmann
force-pushed
the
codex/qiskit-thread-safe-api-initialization
branch
from
August 24, 2026 11:30
be38e9c to
75e6c56
Compare
12 tasks
burgholzer
approved these changes
Aug 24, 2026
burgholzer
left a comment
Member
There was a problem hiding this comment.
LGTM 👍🏼
The PR is still missing tags and a completed checklist. Then this can go in directly 😌
simon1hofmann
added a commit
that referenced
this pull request
Aug 24, 2026
Incorporate the merged one-time Qiskit C API initialization fix from #2223. Assisted-by: GPT-5.6 Sol via Codex Signed-off-by: Simon Hofmann <simon.t.hofmann@tum.de>
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
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
Initialize the vendored Qiskit C API once per versioned translation unit through thread-safe function-local static initialization. This publishes the capsule-backed function table safely when nanobind 3 free-threaded callers create translations concurrently.
Add a focused concurrent import/export regression. This is a prerequisite for #2176 and follows the split-mode/free-threaded binding adoption in #2209.
Testing
AI assistance: Codex assisted with implementation review, isolation, validation, and this description.
Checklist
If PR contains AI-assisted content: