Conversation
fix err
9c85820 to
b6ca547
Compare
|
I think this is ready for initial review despite the failing checks. One is caused by white-space differences in a compile-fail test. I don't know what is causing this. https://github.com/PyO3/pyo3/actions/runs/22745106403/job/65967168100?pr=5856#step:19:3685 There seems to be a bug when |
|
Thank you very much for this! My personal 2cents (don't take it as the PyO3 maintainers opinion): I am not convinced supporting no_std is worth the pain before having a clear idea of what supporting MicroPython C API would entail. Having a branch with an integration MVP (likely with everything outside of the most basic APIs just failing/being UBs) might be worth it before merging this change. Maybe we will discover that MicroPython is too different to make something like PyO3 work and writing a new library would make more sense. |
|
Thanks very much for the PR! I have previously wondered about I'm definitely very open to having micropython supported, I just suspect that having the FFI part working will be much harder than this transformation. I also don't think a micropython support prototype will be blocked by lack of |
|
My primary motivation for this PR is wasm support. I linked to that issue because that's what came up when i searched the issues for |
|
Ah ok, can you clarify the use case where this helps with wasm please? |
CPython officially supports wasm targets. Rust supports wasm targets as well, but parts of |
This PR adds an
stdfeature (enabled by default) and gates some impls behind it as stated in #3510 (comment) .The line count is very high, but it's mostly replacing
stdwithcoreoralloc.Test cases for
PyDictwere usingHashMap, i changed it toBTreeMapbecauseHashMapis instd.The
doctestmodule is gated behindstd.