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

The app gave 20-min and 10-min practices (instead of 20-min twice) #26

Closed
nashiro opened this issue Oct 26, 2023 · 3 comments
Closed

The app gave 20-min and 10-min practices (instead of 20-min twice) #26

nashiro opened this issue Oct 26, 2023 · 3 comments
Assignees

Comments

@nashiro
Copy link

nashiro commented Oct 26, 2023

One pilot participant in the lip condition reported that she was only given 30 min practice on her last day (first session = 20 min, second session = 10 min). I checked the emWave data, which were consistent with her report (20 min + 10 min pulse data).

ID: bde9aded-d803-48bf-a7b0-84dda275e885
Date: 10/23

@noah10
Copy link
Contributor

noah10 commented Oct 27, 2023

I'm seeing very different data. The MindBody.sqlite file shows that she did 40 minutes on 10/22, ending just a few seconds after midnight on 10/23, and no practice at all on 10/23. The last emWave file uploaded has no practice after 10/21. If you have a more recent copy of the emWave file from her laptop please send it to me in Slack.

@noah10
Copy link
Contributor

noah10 commented Oct 27, 2023

Now that I have the up-to-date files, I see 30 minutes on 10/23 in the MindBody.sqlite file and in the emWave file. The reason is that her final segment from 10/22 ended at 10 seconds after midnight on 10/23, which means that the code counted it for 10/23. Theoretically she shouldn't have been allowed to start that final session because you shouldn't be able to begin a session that you don't have time to finish before midnight. I suspect what happened is that it took ~10 seconds for the emWave session to get started after she clicked the "start" button and that is what pushed the session over into the next day. I'm going to open another bug to address that and close this one.

@noah10
Copy link
Contributor

noah10 commented Oct 27, 2023

Closing in favor of #27 .

@noah10 noah10 closed this as completed Oct 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants