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

H10: RR Interval in ms and 1/1024 format. #343

Closed
1 task
Retsam-crypto opened this issue Mar 5, 2023 · 2 comments
Closed
1 task

H10: RR Interval in ms and 1/1024 format. #343

Retsam-crypto opened this issue Mar 5, 2023 · 2 comments
Assignees
Labels
question Further information is requested

Comments

@Retsam-crypto
Copy link

Retsam-crypto commented Mar 5, 2023

Device:

  • Polar H10

What the 1/1024 format means for RRI, please (found it on first page of SDK documentation)?

Silent assumption is that sensor sampling rate is 1000 Hz, RR peaks are discovered from ECG internally, then ECG is down sampled to 130 Hz. But that assumption may be incorrect and sampling rate internally is 1024 Hz. (hence the 1/1024 'format' mentioned in SDK documentation).

Please shed some light on that process.

@Retsam-crypto Retsam-crypto added the question Further information is requested label Mar 5, 2023
@JOikarinen
Copy link
Contributor

Hi @Retsam-crypto,

The format of 1/1024 to represent RR time has been a source of confusion. RRI is the time between the two consecutive R waves in the electrocardiogram (ECG) signal.

The format 1/1024 to represent the RRI time is quite strange. Heart Rate BLE service provides the RR data for HR BLE client in 1/1024 format. I suspect that reason for HR service using the 1/1024 format is the limited resources in some heart rate sensor devices at the time specification was made (approximately 10 years ago).

Why the Polar BLE SDK API exposes the 1/1024 format? Well, it is provided in API to let raw data pass through from heart sensor without any manipulation by the Polar BLE SDK.

However, due to the confusion, the Polar BLE SDK API will be updated in the next release (5.0.0) to only use the "milliseconds" format for RR time. The PolarHrData will have only one property, rrs, instead of rrs and rrsMs. The Android and iOS changes can be viewed in the provided links.

Change in Android: here

Change in iOS: here

@JOikarinen
Copy link
Contributor

It was not a good decision to rename rrs to rrsMs and remove rrsMs, done in Android: here iOS: here

Better idea implemented, let's remove rrs (i.e. format 1/1024) and let's leave only the property rrsMs, done in Android: here iOS: here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants