Qiskit / qiskit
90%
main: 89%

LAST BUILD BRANCH: BD-add-alt-text
DEFAULT BRANCH: main
Repo Added
Files 913
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH gh-readonly-queue/main/pr-12889-9de81195ae997fe8e93e0708daa29b95a2db8842
branch: gh-readonly-queue/main/pr-12889-9de81195ae997fe8e93e0708daa29b95a2db8842
CHANGE BRANCH
x

coverage: 89.739% (-0.003%) from 89.742%
10216870141

push

github

web-flow 
Avoid exporting incorrect `PyInit_*` symbols (#12889)

Using the `#[pymodule]` derive macro in PyO3 0.21 always causes a
`PyInit_*` symbol with a matching name to be exported in the output
`cdylib`.  This is required for the top-level module, in order for
Python to import it---it needs to know which symbol in a shared library
file it should call---but submodules must be manually initialised, so do
not need it.  Including it is typically harmless (and something we've
been doing for a long time), but it is technically against the coding
rules for CPython extensions[^1].

Recent versions of `abi3audit` (0.0.11+) have tightened their symbol
checkers to better match the CPython guidelines, which causes our wheels
to be rejected by their audits.  This is, in theory, not a break of abi3
because CPython could never introduce an API-elvel `PyInit_*` function
themselves without causing problems, so there ought to be no problems
for our users, even with future Python versions. That said, we still
want to pass the audit, because the coding guidelines are useful.

This commit is not the cleanest way of doing things.  PyO3 0.22 includes
a `#[pymodule(submodule)]` option on the attribute macro, which lets us
use all the standard code generation while suppressing the unnecessary
`PyInit_*` symbol.  When we are ready to move to PyO3 0.22, we probably
want to revert this commit to switch to that form.

[^1]: https://docs.python.org/3/c-api/intro.html

42 of 42 new or added lines in 4 files covered. (100.0%)

12 existing lines in 2 files now uncovered.

67336 of 75035 relevant lines covered (89.74%)

388103.32 hits per line

Relevant lines Covered
75035 RELEVANT LINES 67336 COVERED LINES
388103.32 HITS PER LINE
Source Files on gh-readonly-queue/main/pr-12889-9de81195ae997fe8e93e0708daa29b95a2db8842

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
10216870141 gh-readonly-queue/main/pr-12889-9de81195ae997fe8e93e0708daa29b95a2db8842 Avoid exporting incorrect `PyInit_*` symbols (#12889) Using the `#[pymodule]` derive macro in PyO3 0.21 always causes a `PyInit_*` symbol with a matching name to be exported in the output `cdylib`. This is required for the top-level module, in o... push web-flow github
89.74
See All Builds (9023)