-
Notifications
You must be signed in to change notification settings - Fork 95
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move to pyproject.toml #364
Comments
This might get a bit tricky because of the |
Yep that's definitely true. Even an incremental move or moving most things over would be a welcome improvement 🙂 |
Thanks, @Saransh-cpp. 🙏🏽 |
We have the beginnings of a cc @DimitriPapadopoulos (in case this is of interest as well) |
Submitted PR ( #382 ), which moves as much as we can to |
There may be small follow up things that come up. Would suggest raising them in new issues. Thanks all! 🙏 |
It would be good to move to
pyproject.toml
in Numcodecs. In particular it would be nice to add build requirements (like Cython) andsetup_requires
topyproject.toml
.This should also allow us to do things like dropped the vendored C files, which cause us some pain periodically ( #358 (comment) ). Plus drop some of the complexity in
setup.py
.FWIW this was done for Zarr in PR ( zarr-developers/zarr-python#1158 ).
The text was updated successfully, but these errors were encountered: