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

Transition to poetry + address numpy 2.0 and numba 0.60 related issues #134

Merged
merged 17 commits into from
Jun 19, 2024

Conversation

deeenes
Copy link
Member

@deeenes deeenes commented Jun 17, 2024

... because numba fails the function signature check in Py 3.11

@deeenes deeenes mentioned this pull request Jun 17, 2024
@Intron7
Copy link
Contributor

Intron7 commented Jun 17, 2024

Phlya/adjustText#177

@deeenes deeenes changed the title np.arange: do not pass start as kwarg Transition to poetry + address numpy 2.0 and numba 0.60 related issues Jun 18, 2024
@deeenes deeenes requested a review from PauBadiaM June 18, 2024 11:32
@deeenes
Copy link
Member Author

deeenes commented Jun 18, 2024

Hey @PauBadiaM, I think this is ready to merge, I bumped the version number and I would update it also in PyPI to avoid a flood of issues in the upcoming weeks :)

@PauBadiaM
Copy link
Member

Looks fantastic @deeenes! Thanks for taking care of this really. Just one last thing, I see poetry failing for all ubuntu's CI. Maybe you could instead of using the cached version just install it, it should not take long (as it is done in liana-py: https://github.com/saezlab/liana-py/blob/main/.github/workflows/main.yml). But feel free to keep the cached version if you manage to make it work.

@PauBadiaM PauBadiaM merged commit 24ceb3a into main Jun 19, 2024
10 checks passed
@deeenes
Copy link
Member Author

deeenes commented Jun 19, 2024

@PauBadiaM You're right, I removed the caching of poetry itself. It's a mystery why that stopped working in ubuntu-latest.

@PauBadiaM PauBadiaM deleted the arange-numba-fix branch June 20, 2024 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants