search

IanPeake / FatMaxxer

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

Notifications #1

Closed MedTechCD closed 3 years ago

MedTechCD commented 3 years ago

Sound notifications are awkward. Keeps repeating the same words 4-5 times before the actual numbers come through. Maybe because I have the calculation update on 5sec? Notifications on Garmin Edge 830 come through but take up a lot of screen space, see attached screenshot. Even with longer calculation period, the notification stays quite long , hiding the lower half off the screen. Only the first line would be enough, but I don't know if you can decide what comes through. Edge830 Great to see someone is developping an app for real-time alfa1!

IanPeake commented 3 years ago

I saw the sound repeat bug a day or two ago and it was definitely released. But it should be fixed in the most recent version; can you verify?

I will have to support customized notifications---some people have generic wearables with a lot of screen real estate and want all the info in the notifications, whereas others like ourselves just need alpha1 and artifacts.

For now it will be something like "Disable notification content" so that only the title has content.

IanP

On Mon, Jun 14, 2021 at 2:56 AM MedTechCD @.***> wrote:

Sound notifications are awkward. Keeps repeating the same words 4-5 times before the actual numbers come through. Maybe because I have the calculation update on 5sec? Notifications on Garmin Edge 830 come through but take up a lot of screen space, see attached screenshot. Even with longer calculation period, the notification stays quite long , hiding the lower half off the screen. Only the first line would be enough, but I don't know if you can decide what comes through. [image: Edge830] https://user-images.githubusercontent.com/85841362/121815722-14151580-cc78-11eb-907e-7a49293b7f11.jpg Great to see someone is developping an app for real-time alfa1!

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/IanPeake/FatMaxxer/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFSEU6OYCJ5ZP3BXGMCEVXDTSTPKTANCNFSM46T5BAYQ .

MedTechCD commented 3 years ago

Regarding audio notifications, I probably missed an update. Just updated now and they work fine. The option of disabling notification text body is better, but it still takes about 1/3th of the screen because of the blank line. You probably thought about this, but just in case you didn't: Would it be possible to make the app act like some BLE sensor that can be read by the bike computer and displayed as a selectable field? It would then also be saved in the FIT file.

IanPeake commented 3 years ago

Thanks for the feedback. I hadn't thought to try to emulate a BLE sensor---great idea! Would you mind creating new issues for the notification text body blank line and the BLE sensor emulation?

MedTechCD commented 3 years ago

Audio notifications are done. New issue opened for text notifications

MedTechCD commented 3 years ago

Ian,

I created 2 new issues but for some reason Github blocked view of my account. I contacted Github Support to correct this.

Van: IanPeake @.> Verzonden: dinsdag 15 juni 2021 1:57 Aan: IanPeake/FatMaxxer @.> CC: MedTechCD @.>; State change @.> Onderwerp: Re: [IanPeake/FatMaxxer] Notifications (#1)

Thanks for the feedback. I hadn't thought to try to emulate a BLE sensor---great idea! Would you mind creating new issues for the notification text body blank line and the BLE sensor emulation?

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/IanPeake/FatMaxxer/issues/1#issuecomment-861066814 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AUO5LUVRR3XBKABLJYNXDJLTS2JLVANCNFSM46T5BAYQ .