-
Notifications
You must be signed in to change notification settings - Fork 2
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
Read aggregated Embrace Plus and Nowatch data #20
Merged
Merged
Changes from 1 commit
Commits
Show all changes
19 commits
Select commit
Hold shift + click to select a range
022f1d0
refactor: separate raw and aggregated processing
hypebright 23e02da
add `read_aggregated_embrace_plus`
hypebright cfd4f65
make `unzip_files` a utils function
hypebright 743a04d
add unzip utils function
hypebright fe37368
add `read_nowatch` function
hypebright d2fe257
bump version number
hypebright e16479f
refactor: generalize `aggregate_data()` function for multiple devices
hypebright f7e1c08
add classes
hypebright 2b87241
remove comment
hypebright 63d7c27
remove `type` for nowatch
hypebright 25827ea
allow folder input in `read_embrace_plus`
hypebright c367d0a
check activity data in Nowatch object
hypebright d4c9ecd
docs: bump version
hypebright 5f53901
docs: change description
hypebright bcceb75
allow folder as input
hypebright 02a9779
rename more cols
hypebright 5b7e63b
TBD: remove NAs
hypebright 5d6af8a
update `read_and_process_embrace_plus`
hypebright 24ea35d
use MOVE column for ACC
hypebright File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
As you know I'm no expert when it comes to analysing this data, but the
butter_lowpass_filter
doesn't work the same as it does with other devices when there are a lot of NAs. The aggregated Embrace Plus data also has "data" present when the device wasn't worn. When I removed the NAs the flow seemed to work fine again, but would be good to double check what the implications are