-
Notifications
You must be signed in to change notification settings - Fork 217
Enable support for free-threading #1295
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
Conversation
Some other things came up but I'm going to come back to this next week. |
I ran the test suite with I see some tracebacks inside All that to say, I think I agree with #1295 (comment) that we should focus on multithreaded stress tests rather than trying to use the existing test suite as a stress test - we'll likely end up spending most of our effort on fixing races in the test suite itself. For now I'll focus on writing a stress test that uses a thread pool to process many files simultaneously. |
I doubt it'll be very elucidating (it's 1.5 MB...) but here's the full output from running |
d6c78bd
to
d838c82
Compare
dbd34c2
to
404bef2
Compare
Nice! I updated the tracking table too: Quansight-Labs/free-threaded-compatibility#215 |
Enable support for free-threading
This PR:
libcst.native
module as free-threading-compatibleThis depends on #1294 and #1289.
Stack created with Sapling. Best reviewed with ReviewStack.