Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

When listing the alarms to qualify, too many results #131

Open
pmithrandir opened this issue Dec 20, 2023 · 5 comments
Open

When listing the alarms to qualify, too many results #131

pmithrandir opened this issue Dec 20, 2023 · 5 comments
Assignees
Milestone

Comments

@pmithrandir
Copy link
Collaborator

Hello,

On the data sharing page, the alarm listed are too numerous.

For the same seizure, I'll get one line every 5-10 sec... and for each of them I have to go through the process of explaining that it is a seizure or not.

Is there a way to "group" event happenning around the same time ?

@jones139
Copy link
Member

This is a good idea. When I create the anonymised version of the database for publication I group the data into 3 minute periods and use the metadata from the most 'interesting' looking event in that period (either the one that generated a full alarm rather than a warning, or the one with a text description of the event etc.).
Doing something similar on the app would reduce the amount of data the user is presented with, which would help.
One to add to the enhancements list, thanks!

@pmithrandir
Copy link
Collaborator Author

Hi,

Do we have a measure on how many seizure get actually qualified by people ?

The android app is so painful(too many events, sometime multiple occurance...) that I don't even take time to fill them. Usually, I have between 20 and 50 occurrences to fill per day if my son wear the watch, 10 to 20 if it's me.
80% being false alarm, multiplied by 5-10 events per alarm / warning

My understanding is that we want to make the algorithm progress, and we can't do that if we cant' say which data are pointing a seizure and which one is not.

We could either

  • change the API to "group" alerts to be enriched in data sharing view
  • change the app to be able to multi select elements and update them in bulk (bulk must exist as ther is 2 options in the menu)

I'm sadly not qualified with android dev. I could work on the python api if needed.

@jones139
Copy link
Member

jones139 commented Apr 8, 2024

20-50 alarms is excessive - I would say it is not working for you - I'll have a look at some of your data and your settings and see if we can adjust something to reduce the number of false alarms.

I do the grouping in the offline processing to produce the Open Seizure Database, so the simplest would be to improve the data sharing page in the app - I'll have a look and see what i can do with that - either simply allow multiple selections, or maybe group them like I do in the database, which would be better.

@jones139 jones139 self-assigned this Apr 8, 2024
@jones139 jones139 added this to the V4.2 milestone Apr 8, 2024
@pmithrandir
Copy link
Collaborator Author

pmithrandir commented Apr 8, 2024 via email

@jones139
Copy link
Member

jones139 commented Apr 8, 2024

ah, ok, so if we group them into 'events' it should not be so bad - I'll try that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants