-
Notifications
You must be signed in to change notification settings - Fork 86
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
Use PEP 757 C-API to import/export ints #495
Conversation
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #495 +/- ##
==========================================
+ Coverage 85.52% 85.62% +0.09%
==========================================
Files 50 50
Lines 11656 11755 +99
Branches 2202 2222 +20
==========================================
+ Hits 9969 10065 +96
- Misses 1687 1690 +3 ☔ View full report in Codecov by Sentry. |
1fca689
to
697f164
Compare
@casevh, at this point I would appreciate your feedback on capi-workgroup/decisions#35. Proposed API has noticeable speed regression for int->mpz conversion. I'm not sure if this is acceptable for gmpy2. |
2b455a7
to
3ed45cc
Compare
b42a905
to
3612d2f
Compare
3612d2f
to
c9c32c6
Compare
@casevh, this is ready for review |
Thanks for all the work on this. |
Closes #467