[Beta 14] Inventory window mostly missing upon restarting from gamever screen

[Kyzrati]

I don't have many details on this, but there is apparently a very rare issue that can occur upon starting a new run immediately after a loss.

The inventory window does not appear correctly and attempting to interact with items will crash the game. anting reported it on Discord and shared an edited screenshot (attached) to give an idea--basically the inventory is missing its border (a clear indication of having not actually completed its initialization for some reason).

It would be some sort of timing or input-related issue, but no idea beyond that. There's no serious impact on play (other than initial confusion ), since it's right at the beginning of the run and restarting the game will of course return everything to normal, but it would be nice to know exactly how to cause it so it can be resolved. Save data/state would not be helpful here, it's just some specific input steps...

(Edit: I posted this before looking through my data as I try to clean up new issues in Beta 14, and it's looking likely this issue actually did start in Beta 14, and has hit a few other people who didn't report it but are uploading error data. It's likely related to modal inventory in combination with a particular input sequence I haven't been able to repeat yet. Investigating more...)

[Kyzrati]

Whelp spent almost two hours debugging this today and couldn't find the cause, but did learn a bit more about it for now.

It's definitely new in Beta 14, and related to the inventory changes where somehow it's possible for the inventory to not redraw correctly upon restart from game over. Unfortunately the actual crash itself (and therefore the trace data I have) doesn't occur until later when trying to use the inventory, rather than during it's initial partial failure to initialize/draw, so I'm not sure when that's occurring or why. I checked all the related code and didn't come up with anything, so it is probably something extremely convoluted and it'd be nice if we get some more reliable info on a cause. Guess this'll have to remain in there for now until then, but at least it's just a quick restart of the fresh run in that case...