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

Benchmark #18

Merged
merged 25 commits into from
Jun 24, 2024
Merged

Benchmark #18

merged 25 commits into from
Jun 24, 2024

Conversation

ZeroIntensity
Copy link
Owner

This makes PyAwaitable several times faster.

Copy link

netlify bot commented Jun 23, 2024

Deploy Preview for awaitable ready!

Name Link
🔨 Latest commit ab6cd76
🔍 Latest deploy log https://app.netlify.com/sites/awaitable/deploys/66797a47707cdc00088d86cd
😎 Deploy Preview https://deploy-preview-18--awaitable.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@ZeroIntensity
Copy link
Owner Author

@AraHaan could you take a look at this PR and try debugging the test suite on Windows? I'm almost certain the problem is with how raising_callback and raising_err_callback are casted to an awaitcallback, but I'm not sure what's wrong with it - right now, the tests just cast the address (exported through PyModule_AddIntConstant, perhaps the address number is too large on Windows?) to the type through ctypes.cast.

@ZeroIntensity ZeroIntensity merged commit e942937 into master Jun 24, 2024
20 checks passed
@ZeroIntensity ZeroIntensity deleted the benchmark branch June 24, 2024 13:55
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.

1 participant