OpenSeizureDetector / Android_Pebble_SD

The main OpenSeizureDetector Android App, that is published on the Android Play Store.
http://openseizuredetector.org.uk
GNU General Public License v3.0
9 stars 11 forks source link

Problem with Alarms failing after sending SMS alert #147

Closed jones139 closed 1 month ago

jones139 commented 4 months ago

@seaside1 reported: Another thing I noticed is that the Alarm is working if you enable sms location alarm, until it sends the sms, after that it starts failing.

jones139 commented 4 months ago

@seaside1, There is a timer that is intended to prevent too many SMS alerts being sent - after it has sent one it will not send another for at least 1 minute. The audible alarms should still work though - could this be what you have seen?

seaside1 commented 3 months ago

Could very well be that.

seaside1 commented 3 months ago

Alarm works better when sending SMS is on. However as soon as the SMS is sent the Alarms stops sounding. Latch Alarm not working in this context.

jones139 commented 3 months ago

Thanks - I'll check this out - which version of the Android App are you using?

Enabling SMS alerts should not affect alarm generation, but once an SMS is sent, there is a timer which will prevent another SMS being sent within about 1 minute - you should still get the beeping alarm sound from the phone though. I'll test it on V4.2.x which is the current beta testing version.

On Thu, 21 Mar 2024 at 11:33, seaside1 @.***> wrote:

Alarm works better when sending SMS is on. However as soon as the SMS is sent the Alarms stops sounding. Latch Alarm not working in this context.

— Reply to this email directly, view it on GitHub https://github.com/OpenSeizureDetector/Android_Pebble_SD/issues/147#issuecomment-2012025932, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACLSYZGODNTUJEWAWRI2PLYZLAPJAVCNFSM6AAAAABEEHFFS6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJSGAZDKOJTGI . You are receiving this because you authored the thread.Message ID: @.***>

-- Graham Jones http://google.com/+GrahamJones Hartlepool, UK.

jones139 commented 3 months ago

I've just tried it on V4.2.2 but I can't reproduce this issue. I am using the Vivoactive 3 in normal mode, so it is a bit laggy with the communications re-tries, but it sends an SMS, then soon after I get a message about 'SMS already sent, not sending', which is caused by the second ALARM status - but I am still getting ALARM displayed on the phone.

Could you do a video of what you are seeing on the phone please? (it might be best to use low data mode on the Vivoactive 3 so it is more responsive to avoid confusion with the delay).

Thanks Graham.

jones139 commented 3 months ago

I have just made a video of V4.2.2 doing SMS alerts - it is working how it is intended to work - it would be good to compare this to your's. https://photos.app.goo.gl/1FCzLkBSozfjFXsV6

seaside1 commented 3 months ago

Without SMS https://drive.google.com/file/d/1yJKYCmwvxizp7P9wnUewwbl98XJmEgQ0/view?usp=sharing

With SMS enable https://drive.google.com/file/d/1yTvPWdh0ZMZ-9fqKnCAWLWazeybOzA98/view?usp=sharing

seaside1 commented 3 months ago

So to be clear in this case what is not working is the alarm sound. Which in case SMS is enabled is working up to the point when the sms is being sent after that not working. In case SMS is disabled it is not working.

jones139 commented 3 months ago

That is odd - so you are seeing something different to the behaviour in my video? Please try disabling the 'latch alarms' function - I never use that so it is possible there is something wrong with the logic in that function - it did not seem to be working for you because although the text status stayed as 'ALARM' the sound stopped. Also I think I saw an error about failing to send the SMS alert?

jones139 commented 2 months ago

I think that we have resolved the issue with the Latch Alarms not working in V4.2.x (latest release is V4.2.7) - is that true? If so, can we close this issue? Thanks! Graham.

jones139 commented 1 month ago

Latch alarms fixed in V4.2.x which is now the master branch published on play store.