-
Notifications
You must be signed in to change notification settings - Fork 4
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
Feature: Adding support for single threshold non-wear detection across watch platforms #105
base: main
Are you sure you want to change the base?
Conversation
Test of different non wear detection methods
Making std only the default by setting range_criteria = None, if not None, then we use it
Test of different non wear detection methods
Making std only the default by setting range_criteria = None, if not None, then we use it
…/childmindresearch/wristpy into feat/issue-99/non-wear-detetion-single-threshold
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #105 +/- ##
==========================================
+ Coverage 96.34% 96.66% +0.31%
==========================================
Files 11 11
Lines 575 569 -6
==========================================
- Hits 554 550 -4
+ Misses 21 19 -2 ☔ View full report in Codecov by Sentry. |
Adds the range_criteria as a user-defined setting for non-wear detection
Adding tests with user-defined range criteria in config
Probably good for a review but might be best to finalize it after the orchestrator is merged? Or maybe just get some feedback on using config.Settings to pass Optional |
Fix merge conflicts
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor comments, nothing too big.
Moving to single feature non-wear detection for all watches by only using std of acceleration Fixing smoke test as per Freymon's PR comments
Fix docstrings for single criteria
Removing outdated RANGE_CRITERIA
Ruff fix for unused import
Resolves #99
Re-based to #71 , this PR modifies the
detect_nonwear
function to default to using a single thresholdstd_criteria
. This allows the non-wear detection algorithm to work with both .gt3x and .bin files without the need of chaning therange_criteria