FintasticMan

Results 140 comments of FintasticMan

The devcontainer is quite outdated, no-one has gotten around to finishing updating it yet.

I personally think that watch face as separate words is better, and I believe that is mirrored by the code using a capital F in WatchFace.

Do you have a way to reproduce that? I don't think I've noticed that happening, but it sounds like a threshold issue. To check you could make the values in...

Now you mention it, I think I have noticed something similar happening. I think that then there isn't necessarily an issue with lower to sleep and rather raise to wake....

I've been running this myself, and it's been working exactly how I expect it to. It already has a checkbox in the Wake Up settings submenu, so I think it...

Rebased onto latest develop.

Did you test it with #826 and #648? Those both improve the experience a lot, so that even though it does turn off if you shake it, it turns back...

@Riksu9000 I think there might be an issue with the clang-format GitHub actions you made. It says there's an issue with the formatting, but it doesn't upload any patches and...

@Riksu9000 It does seem to be working, but the issues it found aren't with my changes in particular. If that's how it's supposed to work, that's fine. I do think...

Thanks for the review! The reason I chose to keep the `y` variable as well as the circular buffer is because it saves a modulo operation for accessing just the...