-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Merge in upstream MicroPython changes #2999
Comments
A small bump on this, as it's something I'd really love to see happen. Notable things that we'd get:
I don't presently have the time (or expertise) to do this myself, but I would be interested in sponsoring this. |
I am willing to do this when I have time. We are thinking of waiting for the MicroPython 1.13 release. |
Bump from me too! MicroPython 1.13 was released in September, and the walrus assignment operator is very handy for tidy code. |
+1 on this from me, being able to write native modules without needing to integrate with the core would be huge. |
See https://github.com/pimoroni/pimoroni-pico/tree/main/micropython/modules for a use case for this. This probably ties also into #2825 |
@timonsku having been through the process of trying to use that, I have to say it's not as useful as it seems. The problem is that all the functions you are going to call have to be included in a special table at compile time — and only a handful is there by default — so while it's useful for speeding up some purely computational functions, it's very limited otherwise. |
I'm also very concerned about the support burden of native modules that work for ARM but not xtensa or risc-v for example. I'd encourage folks to write native modules in the core instead. We're happy to have them. |
Is this separate issue still valuable alongside #4280? |
Ya, I think so. Issues can live when inactive and PRs should be closed when they are. We can close this when the 1.14 merge is done. |
It automates ignoring parts of MP we don't use. Related to #2999
nice! |
No description provided.
The text was updated successfully, but these errors were encountered: