Damien George
Damien George
I've made a few comments, mostly regarding coverage. About the `MICROPY_PY_TSTRING_MAX_xxx` constants: are these really needed? What's wrong with just letting the t-strings grow as much as the user needs...
@koxudaxi thank you for working on this template string implementation. What is the current status from your side? If it's ready for a review then please rebase on latest master...
@koxudaxi if you rebase on latest master, it'll fix the zephyr CI.
@koxudaxi I had a good look at this PR and how it parses t-strings. From what I understand, it creates a new nested parser when it encounters a t-string, in...
See #890 for a patch for this.
It looks like the issue occurs when using PSRAM on an S2/S3 board. Because USB DMA cannot read from PSRAM, so the data sent to the host ends up as...
Should be fixed by #1034 (which is merged).
If you look here: https://github.com/micropython/micropython-lib/actions/runs/16717239939/job/47313198532?pr=1040 it shows you the problems. Basically you need to put a few things on one line, instead of over many lines.
Thanks for this, it's a good addition. I did myself want to tackle this issue of emitting vs loading native code. The way you approached it with `MICROPY_LOAD_NATIVE_MODULES` could probably...
> i have esp-idf (ESP-IDF v5.2.6-224-g72d06017df) Please try using a more recent IDF version. I suggest IDF 5.5.1.