Add ability to apply sensitivity presets - #108
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
| * Applications should leave any data in here unchanged when setting the Config. | ||
| */ | ||
| padding: uint8_t[49], | ||
| padding: uint8_t[44], |
There was a problem hiding this comment.
Huge mystery here... even though the struct appears to be identical to the one in python and the official SDK, for some reason the struct ended up being 5 bytes bigger than intended (255) bytes. I simply reduced the declared padding here to fix it for now, but... 😕
There was a problem hiding this comment.
hmmm I wonder if something is being converted improperly to be bigger than they need to be? or is it literally just exactly the same but for some reason padding is bigger?
There was a problem hiding this comment.
i'm willing to guess it's a bug in the struct package i put together for this. i manually counted everything by eye and it at least matched the official c struct
There was a problem hiding this comment.
hmm my quickest guess would be maybe something with regards to the enabled_sensors thing we made. very simple custom type that conveniently packs and unpacks 9 panels worth of four booleans each into 5 bytes where maybe it's missing just one of the 5 bytes somehow? but I guess if everything before the padding is the same, then confusion. Maybe there's a bug in how many bytes it thinks it's written already?
fchorney
left a comment
There was a problem hiding this comment.
Aside from weird padding issue, LGTM! Works on my pads!
|
Merging this, more work will continue in #127 |
Adds buttons to apply high/normal/low sensitivity settings to the pad matching the same ones found in the current (2025) version of the official app. We can follow this up with more UI updates to allow fine-tuning sensitivity per panel/sensor.
Also adds better support for live plug/unplug of stages while the page remains open.