childmindresearch / wristpy

https://childmindresearch.github.io/wristpy/
GNU Lesser General Public License v2.1
5 stars 5 forks source link

Feat/issue 102/handle idle sleep mode #143

Open Asanto32 opened 6 days ago

Asanto32 commented 6 days ago

Support to handle the idle_sleep_mode for Actigraph watches. This function fills in time gaps in the data created by enabling idle_sleep_mode. In cases when the samples are not linearly spaced within one second, the data is resampled.

codecov[bot] commented 6 days ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.41%. Comparing base (d0e9226) to head (a5a5d5d).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #143 +/- ## ========================================== + Coverage 96.25% 96.41% +0.15% ========================================== Files 11 12 +1 Lines 561 586 +25 ========================================== + Hits 540 565 +25 Misses 21 21 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Asanto32 commented 9 hours ago

This is ready for review now. Small chance of merge conflicts with the orchestrator compared to the new one with full directory processing but I tried to make sure it should work together.

Note: when there are significant amounts of idle_sleep_mode gaps it does slow down processing. Not due to the idle_sleep_mode_imputation (which only takes 5-10s), but from the addition of significantly more data points (example test case on my computer adds in more than 50% of the data thus increase in processing time by ~50%)