Using displayio.Tilegrid.transpose_xy will crash the MAGTAG sending it to Safe Mode

[jposada202020]

This was tested with the following code in a MagTag using CircuitPython 6.1.0

import displayio
import terminalio


font = terminalio.FONT
glyph= font.get_glyph(ord('A'))
letter = displayio.TileGrid(glyph.bitmap, pixel_shader=displayio.Palette(1), default_tile=glyph.tile_index, tile_width=glyph.width, tile_height=glyph.height)
letter.transpose_xy = True

This works fine in a wioterminal

[dhalbert]

Could you try this with 6.2.0-beta.2?

[FoamyGuy]

I tested this on:

Adafruit CircuitPython 6.2.0-beta.2 on 2021-02-11; Adafruit MagTag with ESP32S2

I'm not getting safe mode, but the device does "hard" crash and reboot.

[rsbohn12]

Tested on

Adafruit CircuitPython 6.1.0 on 2021-01-21; Adafruit Metro ESP32S2 with ESP32S2

-- hard crash and reboot.

[microdev1]

I am getting a hard crash and a subsequent reset into safe-mode with the following setup :-

Adafruit CircuitPython 6.2.0-beta.2 on 2021-02-11; microS2 with ESP32S2

I ran a backtrace and here are the results :-

/home/mdev/code/circuitpython/ports/esp32s2/../../shared-module/displayio/TileGrid.c:111
/home/mdev/code/circuitpython/ports/esp32s2/../../shared-module/displayio/TileGrid.c:307
/home/mdev/code/circuitpython/ports/esp32s2/../../shared-bindings/displayio/TileGrid.c:280
/home/mdev/code/circuitpython/ports/esp32s2/../../py/objfun.c:86
/home/mdev/code/circuitpython/ports/esp32s2/../../py/runtime.c:624
/home/mdev/code/circuitpython/ports/esp32s2/../../py/runtime.c:1169
/home/mdev/code/circuitpython/ports/esp32s2/../../py/vm.c:425
/home/mdev/code/circuitpython/ports/esp32s2/../../py/objfun.c:286
/home/mdev/code/circuitpython/ports/esp32s2/../../py/runtime.c:624
/home/mdev/code/circuitpython/ports/esp32s2/../../py/runtime.c:598
/home/mdev/code/circuitpython/ports/esp32s2/../../lib/utils/pyexec.c:114
/home/mdev/code/circuitpython/ports/esp32s2/../../lib/utils/pyexec.c:524
/home/mdev/code/circuitpython/ports/esp32s2/../../main.c:530
/home/mdev/code/circuitpython/ports/esp32s2/../../main.c:597
/home/mdev/code/circuitpython/ports/esp32s2/supervisor/port.c:330
/home/mdev/code/circuitpython/ports/esp32s2/build-microdev_micro_s2/esp-idf/../../esp-idf/components/esp32s2/cpu_start.c:453
/home/mdev/code/circuitpython/ports/esp32s2/build-microdev_micro_s2/esp-idf/../../esp-idf/components/freertos/xtensa/port.c:143

[jposada202020]

Aditional comments

For 6.1.0 goes to safe mode

Adafruit CircuitPython 6.1.0 on 2021-01-21; Adafruit MagTag with ESP32S2
>>> import test_transpose

[tio 12:42:33] Disconnected
[tio 12:42:36] Connected
Auto-reload is off.
Running in safe mode! Not running saved code.

You are in safe mode: something unanticipated happened.
CircuitPython core code crashed hard. Whoops!
Crash into the HardFault_Handler.
                                 Press any key to enter the REPL. Use CTRL-D to reload.

For 6.2.0 we have this in the terminal. Hard crash and Reboot

    Adafruit CircuitPython 6.2.0-beta.2 on 2021-02-11; Adafruit MagTag with ESP32S2
    >>> import test_transpose
    [tio 12:48:38] Disconnected
    [tio 12:48:41] Connected

[deshipu]

Does it happen when you set the width, height, tile_width and tile_height explicitly to constant numbers?

[deshipu]

I think I see the problem:
https://github.com/adafruit/circuitpython/blob/main/shared-module/displayio/TileGrid.c#L107

The width variable is defined as int16_t but the self->pixel_width it is getting assigned to is uint16_t.

[jposada202020]

Does it happen when you set the width, height, tile_width and tile_height explicitly to constant numbers?

Yes, this code will produce crash and restart.

import displayio
import terminalio

font = terminalio.FONT
glyph= font.get_glyph(ord('A'))
letter = displayio.TileGrid(glyph.bitmap, pixel_shader=displayio.Palette(1), default_tile=33, tile_width=6, tile_height=14)
letter.transpose_xy = True

[deshipu]

Can you try #4287 and see if it fixes it?