-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Adafruit-Qualia-S3 "pystack exhausted" #8574
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
Comments
This is the same behavior that led me to open #8573. |
I found the "breaking version". This version allows me to Adafruit CircuitPython 9.0.0-alpha.2-16-gccd667d97a on 2023-11-06; Adafruit-Qualia-S3-RGB666 with ESP32S3 This subsequent version does not: Adafruit CircuitPython 9.0.0-alpha.2-17-g01be5f402e on 2023-11-08; Adafruit-Qualia-S3-RGB666 with ESP32S3 |
@jacobmarble Try placing |
I tried CIRCUITPY_HEAP_START_SIZE=1024000 with adafruit-circuitpython-adafruit_qualia_s3_rgb666-en_US-20231109-3ca9802.uf2 and it still gave pystack exhausted. I tried adafruit-circuitpython-adafruit_qualia_s3_rgb666-en_US-20231106-ccd667d.uf2 and this worked for me. This worked for all four example programs. |
You should try both the CIRCUITPY_PYSTACK_SIZE and the CIRCUITPY_HEAP_START_SIZE parameters. I tested the adafruit_qualia library and setting those two parameters allowed the library to load with the latest build version. |
Do you still have the pystack size change with the old version? I think there are two issues here:
|
I commented the |
Ok, that's interesting. Maybe I broke the pystack size with the split heap PR. Still worth a separate issue (this one.) |
Using these suggested params in settings.toml FWIW I did the following: factory reset soft reboot Auto-reload is on. Simply save files over USB to run them or enter REPL to disable. Code done running. NOTE: line 23 is as follows: |
That's progress 😁 try doubling the heap start to 2048000 |
Steps:
|
I think this is helpful information, especially since your example is pretty easy to reproduce. I've been using a START_SIZE of 3072000 on boards that have enough RAM and 1536000 on the boards with less. |
Original code used uint32_t* so `/ sizeof(size_t)` was needed. It is a uint8_t* now so that division makes it 4x smaller. Whoops! Fixes #8574
CircuitPython version
Code/REPL
Behavior
Description
Based on an older issue, I tried setting
CIRCUITPY_PYSTACK_SIZE=4000
insettings.toml
. The result ofimport adafruit_qualia
is then:Additional information
The text was updated successfully, but these errors were encountered: