Skip to content

[BUG]: cython files packaged as part of the wheel #766

Closed

Description

@ZzEeKkAa

Is this a duplicate?

Type of Bug

Something else

Component

cuda.bindings

Describe the bug

After installing wheel I have cpp, pxd, pyx files in the module install folder, however those are not needed to be a part of wheel package. Fragment from RECORD file:

cuda/bindings/_internal/nvjitlink.cpp,sha256=KXw9hS9ZRSeEcL1kJeksSsWIlkndKAtCAHg7l28tqa4,497706
cuda/bindings/_internal/nvjitlink.cpython-311-x86_64-linux-gnu.so,sha256=ydsDbbnSHaFrbeBFqn4r8IX_nBeCpd2oFBTiTKJuIu0,78144
cuda/bindings/_internal/nvjitlink.pxd,sha256=j0NrfgZYIeZ68_X7i2yoP1kpdIPSXk6mKHI8kZstzz4,2435
cuda/bindings/_internal/nvjitlink.pyx,sha256=kAS0RP2J7wczkWNHEphsvry7TsVg6JzARovPUIGTRIc,15947
cuda/bindings/_internal/nvjitlink_linux.pyx,sha256=kAS0RP2J7wczkWNHEphsvry7TsVg6JzARovPUIGTRIc,15947
cuda/bindings/_internal/nvjitlink_windows.pyx,sha256=J8TvJ00LYlBdnQrkmYvix5pG1iJXd8qApz7ogkpVK90,15253

How to Reproduce

Install wheel from pypi

Expected behavior

Only .so files should end up in the wheel. .pxd and .pyx should end up in sdist. .cpp should be ignored in both cases.

Operating System

No response

nvidia-smi output

No response

Activity

  1. leofang commented on Jul 19, 2025

    @leofang
    Member

    "It's not a bug, it's a feature!" 😆

    Joke aside, as discussed in an internal thread, this happens to several Python projects too. We should purge source files from wheels indeed (except for .pxd files).

    xref cupy/cupy#9226

  2. added
    enhancementAny code-related improvements
    triageNeeds the team's attention
    P1Medium priority - Should do
    packagingAnything related to wheels or Conda packages
    cuda.bindingsEverything related to the cuda.bindings module
    cuda.coreEverything related to the cuda.core module
    and removed
    bugSomething isn't working
    on Jul 19, 2025
  3. added theissue type on Oct 13, 2025
  4. ZzEeKkAa commented on Feb 5, 2026

    @ZzEeKkAa
    ContributorAuthor

    Correction to the original issue: .pxd files must be presented in the wheel, so they can be later reused.

    The issue was partially targeted here: #1578

  5. Andy-Jost commented on Feb 9, 2026

    @Andy-Jost
    Contributor

    .pxi files are also needed (included from .pxd)

  6. modified the milestones: cuda.bindings backlog, on Feb 10, 2026
  7. moved this from Todo to Done in CCCLon Feb 11, 2026
  8. modified the milestones: , cuda.bindings next on Feb 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

P1Medium priority - Should docuda.bindingsEverything related to the cuda.bindings modulecuda.coreEverything related to the cuda.core moduleenhancementAny code-related improvementspackagingAnything related to wheels or Conda packages

Type

Projects

  • Status
    Done

Relationships

None yet

Development

No branches or pull requests

Issue actions