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

Test hictkpy against Python 3.13 #99

Merged
merged 11 commits into from
Oct 22, 2024
Merged

Test hictkpy against Python 3.13 #99

merged 11 commits into from
Oct 22, 2024

Conversation

robomics
Copy link
Contributor

@robomics robomics commented Oct 20, 2024

Partial progress towards solving #84.

hictkpy builds without problems on Python 3.13, and given that we do not link to any external libraries at runtime, we could in principle build wheels for Python 3.13 and let users deal with third-party deps that do not have wheels for Python 3.13 (currently pyarrow).

However, in the remote case where pyarrow==18 public Python API changes in a way that breaks hictkpy, we would be left with a version of hictkpy that installs fine, but fails at runtime when using Python 3.13.

Thus, I think it is better to err on the safe side and ship hictkpy v1.0.0 without wheels for Python 3.13.
Then, release v1.0.1 at a later stage, when pyarrow==18 becomes available, with pre-built Python 3.13 wheels.

@robomics robomics added dependencies Pull requests that update a dependency file CI Updates to the Continuous Integration pipelin labels Oct 20, 2024
@robomics robomics changed the title Support Python 3.13 Test hictkpy against Python 3.13 Oct 22, 2024
robomics added a commit that referenced this pull request Oct 22, 2024
@robomics robomics marked this pull request as ready for review October 22, 2024 10:47
@robomics robomics merged commit 628d8cf into main Oct 22, 2024
1 of 2 checks passed
@robomics robomics deleted the ci/py313 branch October 22, 2024 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Updates to the Continuous Integration pipelin dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant