Skip to content

Respiratory and heart rate detection#405

Merged
psomhorst merged 24 commits intodevelopfrom
feature/211_rr_hr_detection_psomhorst
Aug 11, 2025
Merged

Respiratory and heart rate detection#405
psomhorst merged 24 commits intodevelopfrom
feature/211_rr_hr_detection_psomhorst

Conversation

@psomhorst
Copy link
Copy Markdown
Contributor

This PR adds the respiratory and heart rate detection algorithm.

Compared to earlier versions, this version has a feature to refine the rate based on parabolic interpolation. It shifts the estimated maximum value towards the left or right of the maximum based on the relative height of the points directly to the left and right.

The algorithm includes a plotting feature with examples.

The PR includes a notebook. I will test adding the notebook to the documentation directly.

Tests are written based on available data, as well as simulated data, which tests many combinations of heart and respiratory rates. Note that more tests were run locally, but a smaller set of simulations was committed to reduce testing time on the server.

@psomhorst psomhorst force-pushed the feature/211_rr_hr_detection_psomhorst branch 5 times, most recently from 7002199 to 3f8c393 Compare August 7, 2025 19:28
@psomhorst psomhorst force-pushed the feature/211_rr_hr_detection_psomhorst branch from 3f8c393 to 9f184a7 Compare August 8, 2025 09:21
@psomhorst
Copy link
Copy Markdown
Contributor Author

psomhorst commented Aug 8, 2025

@JulietteFrancovich I have addressed your comments by improving the documentation, improving the warning message, and adding annotations with the estimated rates to the plots.

@psomhorst psomhorst force-pushed the feature/211_rr_hr_detection_psomhorst branch from 9c67730 to 5bf3852 Compare August 8, 2025 10:54
@psomhorst psomhorst merged commit bf8a284 into develop Aug 11, 2025
3 checks passed
@psomhorst psomhorst deleted the feature/211_rr_hr_detection_psomhorst branch August 11, 2025 11:23
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.

2 participants