Skip to content

Commit

Permalink
Declaring required dependencies for wheel package (#2283)
Browse files Browse the repository at this point in the history
The wheel package misses `requires_dist` entry in package info, but dpnp
package has a decency on numpy and dpctl packages.
The PR proposes to declare required dependencies in `setup.py`.

Thus the package info of dpnp wheel package built in public CI will
include:
```bash
requires_python: >=3.9,<3.14
requires_dist: ['dpctl>=0.19.0dev0', 'numpy']
```
and for internal CI:
```bash
requires_python: >=3.9,<3.14
requires_dist: ['dpctl>=0.19.0dev0', 'numpy', 'dpcpp-cpp-rt', 'onemkl-sycl-blas', 'onemkl-sycl-dft', 'onemkl-sycl-lapack', 'onemkl-sycl-rng', 'onemkl-sycl-stats', 'onemkl-sycl-vm']
```

Note that this PR also specifies a requirement on supported range of
python versions.
  • Loading branch information
antonwolfy authored Jan 28, 2025
1 parent 83fef36 commit df307dd
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,7 @@ def _get_cmdclass():
Topic :: Software Development
Topic :: Scientific/Engineering
Operating System :: Microsoft :: Windows
Operating System :: POSIX :: Linux
Operating System :: POSIX
Operating System :: Unix
"""
Expand Down Expand Up @@ -82,4 +83,6 @@ def _get_cmdclass():
]
},
include_package_data=False,
python_requires=">=3.9,<3.14",
install_requires=["dpctl >= 0.19.0dev0", "numpy"],
)

0 comments on commit df307dd

Please sign in to comment.