search

IanPeake / FatMaxxer

Android Project to find FatMax in real time with a Polar H10
Other
61 stars 11 forks source link

FatMaxxer hangs at 01:50 #29

Open afxmedia opened 3 years ago

afxmedia commented 3 years ago

Version: 210707_2112.main.0453da4 Issue: sometimes it hangs at exactly after 1min 50 seconds. Once it does, the only way to stop it from happening was to "Clear Data" from app storage. Device & OS: Samsung Note 9, Android 10

Debug log attached: ftmxr_20210722104629.debug.log

IanPeake commented 3 years ago

Reproducible in 210707_1422.main.a3a6abe

afxmedia commented 2 years ago

issue persists in Version 220110_1923.main.9a09a60 when "Enable ECG artifact logging" is turned on.

winice-test commented 1 year ago

@IanPeake I did some investigation by print. It seems that line 2707-2711 make some difference in Polar SDK library. The appendData in these DataPoint seems to should be increasing order. The first call at 1:50 which is less than 2:00. Then it reports an error like:

BleHrClient/Stop observing HR BleGattBase/Remove notification characteristic for 00002a37-0000-1000-8000-00805... Error: new x-value must be greater then the last value. x-values has to be order...

I tried to comment out these 5 appendData. Then it seems to pass 01:50 with ECG artifact logging enabled.

I am not familar with graphview and Polar SDK. Would you please continue to take a look. Thank you.

ensingerphilipp commented 1 year ago

Bug still persists with current state of main repo

IanPeake commented 1 year ago

Thanks Phillip, What hardware/Android are you using?

Ian

On Tue, 20 Jun. 2023, 19:10 Philipp Ensinger, @.***> wrote:

Bug still persists with current state of main repo

— Reply to this email directly, view it on GitHub https://github.com/IanPeake/FatMaxxer/issues/29#issuecomment-1598404774, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFSEU6O6JIAKFGN5GUMSS3DXMFSGXANCNFSM5AZSTA7A . You are receiving this because you were mentioned.Message ID: @.***>

IanPeake commented 1 year ago

Thanks @winice, I will take a look. It may take me a few days to get back into the detail.

Ian

On Fri, 25 Nov. 2022, 18:31 winice, @.***> wrote:

@afxmedia https://github.com/afxmedia I did some investigation by print. It seems that line 2707-2711 make some difference in Polar SDK library. The appendData in these DataPoint also impacts the DataPoints come from Polar SDK. Then Polar SDK reports an error like

BleHrClient/Stop observing HR BleGattBase/Remove notification characteristic for 00002a37-0000-1000-8000-00805... Error: new x-value must be greater then the last value. x-values has to be order...

I tried to comment out these 5 appendData. Then it seems to pass 01:50 with ECG artifact logging enabled.

I am not familar with graphview and Polar SDK. Would you please continue to take a look. Thank you.

— Reply to this email directly, view it on GitHub https://github.com/IanPeake/FatMaxxer/issues/29#issuecomment-1327109405, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFSEU6ONBYMYXTR5XG2DE5LWKBTNHANCNFSM5AZSTA7A . You are receiving this because you commented.Message ID: @.***>

ensingerphilipp commented 1 year ago

Thanks Phillip, What hardware/Android are you using? Ian … On Tue, 20 Jun. 2023, 19:10 Philipp Ensinger, @.> wrote: Bug still persists with current state of main repo — Reply to this email directly, view it on GitHub <#29 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFSEU6O6JIAKFGN5GUMSS3DXMFSGXANCNFSM5AZSTA7A . You are receiving this because you were mentioned.Message ID: @.>

Hi there! I am using a Samsung S21 Ultra with Android 13.