search

EmotionCognitionLab / lipbelly

MIT License
0 stars 0 forks source link

Don't delete emWave sessions > 4m30s #22

Closed noah10 closed 9 months ago

noah10 commented 11 months ago

The code currently deletes emWave sessions shorter than five minutes. Since most sessions are supposed to be 20 minutes, this is only a problem in one place: The five minute rest and paced breathing sessions that we do at setup, which can be deleted if they are just a hair too short. Change this limit to 4m30s so we don't delete the legitimate five minute rest and paced breathing sessions.

noah10 commented 11 months ago

We should also set the average coherence reporting minimum to 4.5 minutes so that we capture the average coherence for the setup breathing.

choipaul92 commented 10 months ago

Can this be tested using Mind-Body 0.0.11? @noah10

choipaul92 commented 9 months ago

@noah10

11/28 [1.0.2 dev version]:

-1st 5 minute baseline > quit the app with 1 minute remaining (the 4 minute session was not saved in emwave) -I relaunched the app and was given the 5 minute baseline again > quit the app with 20 seconds remaining (the 4 minute 40 second was not saved was no saved in emwave) -I relaunched the app and was given the 5 minute baseline again > quit the app with 10 seconds remaining (the 4 minutes 50 seconds was not saved in emwave)

noah10 commented 9 months ago

This sounds like it was an invalid test. Rather than quitting the app, you'll need to remove your ear sensor and let the session completely time out. You should see the message "The session has ended because it has been a long time since the device detected a pulse signal. Please press the start button when you are ready to start over." Once you see that, go into emwave to confirm that (a) the session has been saved and (b) that it is longer than 4m30s. Then go back to the app, start a new session and complete it. Once the app has uploaded your data, go back into emwave and confirm the first session is still there.

choipaul92 commented 9 months ago

Data is saved on emwave. The Baseline data was tested via removing my ear sensor and letting the session completely run out. We also tested (a) and (b) together

Baseline data: -5 min 02 seconds -4 min 36 seconds -5 min

20 minute Session data: -20 min -20 min

noah10 commented 9 months ago

OK - the 4m36s saved session sounds like this is working.