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

Add reader for SHHS #48

Merged
merged 6 commits into from
Dec 8, 2021
Merged

Add reader for SHHS #48

merged 6 commits into from
Dec 8, 2021

Conversation

hofaflo
Copy link
Collaborator

@hofaflo hofaflo commented Dec 7, 2021

SHHS

This is pretty similar to read_mesa, except for the following differences:

  • SHHS is grouped into two "visits" (there are two recorded nights for some participants), therefore there is an additional directory level (shhs1/shhs2).
  • For MESA the ECG channel name is "EKG", for SHHS it's "ECG".
  • SHHS does not include actigraphy data (while read_mesa doesn't read that data yet, this is planned).

Due to the similarity, large parts of read_mesa and read_shhs could be put into a new function. I'd like to postpone refactoring this until read_mesa handles actigraphy data and both functions handle metadata.

@cbrnr cbrnr merged commit aeba268 into cbrnr:main Dec 8, 2021
@cbrnr
Copy link
Owner

cbrnr commented Dec 8, 2021

Thanks @hofaflo!

@hofaflo hofaflo deleted the read-shhs branch December 8, 2021 11:10
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

Successfully merging this pull request may close these issues.

None yet

2 participants