-
Notifications
You must be signed in to change notification settings - Fork 7
Fix for error "Unsupported PyUnicodeObject kind: 0" #76
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
base: master
Are you sure you want to change the base?
Conversation
|
Hey, @Kuinox! Which CPython version and OS/CPU platform are you using? Could you please share a script that reproduces the problem? Thank you! |
|
Ah yes sorry. I'll try to make a reproducible script. |
|
Thank you! I have not tried running this code on Windows, so it's very likely that it never worked, indeed. I was just surprised to see that you had to implement support for "legacy" strings in the PR (which should be long gone?). I would like to fix the tests on the main branch first and return to this a bit later. Thanks for flagging the issue! |
|
I did try to reproduce it yesterday, but I couldn't. OTOH, if you need me to extract info from the dump, I can do it. |
|
It fails with |
|
Hmm, interesting... All the tests pass on Linux with different CPython versions. I'll see if I can get it running on Windows. |
|
You need a lib that create theses old string, personally it was a dump in middle of boto3 listing files on S3. |
When I tried to get a backtrace of my script, i got the error
Unsupported PyUnicodeObject kind: 0I delegated the fix to Codex CLI, it produced this patch, which allow me to get a python backtrace.
I did not reviewed the fix, I'm opening this PR if the author is interested in the fix itself, or for users if the bug is blocking them, this patch could help.
Feel free to close this PR.