Skip to content
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

Convert to pyproject style #167

Merged
merged 52 commits into from
Jul 11, 2024

Conversation

catanzaromj
Copy link
Contributor

@catanzaromj catanzaromj commented Jul 6, 2024

This PR will update ripser.py to use the pyproject.toml spec, as well as unpin dependencies so that it is compiled with newer versions of packages by default (e.g., numpy 2.0.0).

EDIT: When I updated the CI builds, I forgot to include robinhood in with the wheels. I added the robinhood clone back in so the next release will push wheels to PyPI with the robinhood speedup.

@catanzaromj catanzaromj self-assigned this Jul 6, 2024
@catanzaromj
Copy link
Contributor Author

Sorry for the load of commits--I accidentally used an old branch. I'll be sure to squash them before merging.

@catanzaromj catanzaromj requested a review from ctralie July 6, 2024 16:05
@catanzaromj catanzaromj marked this pull request as draft July 6, 2024 18:50
@catanzaromj
Copy link
Contributor Author

I just realized that I removed the robinhood clone when I fixed the CI build stuff. The most recent release (0.6.8) was not compiled with robinhood. When this version is released, robinhood will be re-included.

@catanzaromj catanzaromj marked this pull request as ready for review July 6, 2024 19:51
@catanzaromj catanzaromj merged commit 64c7517 into scikit-tda:master Jul 11, 2024
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
1 participant