search

InfiniTimeOrg / InfiniTime

Firmware for Pinetime smartwatch written in C++ and based on FreeRTOS
GNU General Public License v3.0
2.74k stars 938 forks source link

Unresponsive #2110

Closed tituscmd closed 1 month ago

tituscmd commented 2 months ago

Latest CI build feels less responsive to me. I usually know where and when to swipe by muscle memory but it feels like I have to wait longer between each swipe since updating.

mark9064 commented 2 months ago

Can you bisect? If not, which build are you comparing to? Edit: also, any particular transitions that feel worse (start screen, end screen, direction)? If it's to a watchface do say which one

tituscmd commented 2 months ago

I'm comparing to a build of mine from about 2 months ago. Any movement feels less responsive, such as swiping through the pages of the app loader or swiping down the ringing timer.

mark9064 commented 2 months ago

Do you have a commit ref for that build?

tituscmd commented 2 months ago

Unsure where to find that, but I have the DFU on hand.

mark9064 commented 2 months ago

When the firmware is running you can see the short ref in the settings/about screen

tituscmd commented 2 months ago

I see, thank you! The short ref is d6e9361a

mark9064 commented 2 months ago

That ref is to the tip of your casio weather branch (https://github.com/tituscmd/InfiniTime/tree/casio_weather) I think you might have the wrong ref here, as you said the build is about 2 months old, but the tip of this branch is only 3 days old

tituscmd commented 2 months ago

Oh, I gave you the wrong one. This is it: 57e625d4 The one I gave you is the one that feels unresponsive to me.

mark9064 commented 2 months ago

OK great, now we have a range of commits which your regression lies inside. I'll have a look to see if I have any ideas as to what it could be Edit: compare https://github.com/InfiniTimeOrg/InfiniTime/compare/57e625d...tituscmd:casio_weather

mark9064 commented 2 months ago

Would you be willing to try a few different builds to see if we can narrow down the cause? Edit: the older CI builds have expired so you would have to build them yourself unfortunately

tituscmd commented 2 months ago

I don't have access right now to a way to flash firmware, since the InfiniLink Beta has expired. Once that's back again, I can do that for sure!

mark9064 commented 2 months ago

Sounds good! I think some commits worth testing are

If you have access to any laptop (windows/macos/linux) you can also flash using that (assuming it has bluetooth), but no worries at all if you prefer to stick with your phone. Setting up a laptop to flash will probably take 20 mins of setup if it isn't linux

tituscmd commented 2 months ago

Is it possible on windows?

mark9064 commented 2 months ago

Yes, I've previously written a DFU Python script that is cross platform, I don't mind sending it here. The setup is manual though. And as long as your PC has a bluetooth adapter of course!

tituscmd commented 2 months ago

I think tho it's unneccessary because to do any coding with the PineTime I would boot over to linux anyway, so I think I'm good on that behalf! But I have better experience with flashing via InfiniLink, so I think I'm gonna wait for that.

tituscmd commented 1 month ago

I'm not sure what's different this time, but today I flashed d6e9361a onto my main PineTime and it feels much smoother than my previous version! I don't know what's the difference to last time I reported on this, but I'm certainly not complaining and this issue is as good as solved in my opinion.

Previous version (feels less smooth) https://github.com/user-attachments/assets/f661472f-8c37-4cc1-b20d-f998b2fa4887

Current version (feels smoother) https://github.com/user-attachments/assets/854fe479-0047-49d0-bff0-da4eec2a0d9f