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

Replace required Cython dependency with optional Numba #53

Merged
merged 5 commits into from
Jan 10, 2021
Merged

Conversation

sethaxen
Copy link
Collaborator

Having Cython as a required dependency puts constraints on this package. For example, we have to build wheels for multiple systems. We're only using it to accelerate a single function, so this PR replaces instead uses Numba if available.

@sethaxen sethaxen merged commit 3ec0e47 into master Jan 10, 2021
@sethaxen sethaxen deleted the numba branch January 10, 2021 12:25
@coveralls
Copy link

Coverage Status

Coverage increased (+0.8%) to 59.041% when pulling ddf06cb on numba into eac3c80 on master.

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.

None yet

2 participants