-
Notifications
You must be signed in to change notification settings - Fork 1.3k
adaptive heart rate monitor app #4145
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
base: master
Are you sure you want to change the base?
Conversation
Added metadata for AHRM application including details such as version, description, tags, and supported platforms.
|
This looks really cool! One question - does this modify the existing heart rate function, so if another app calls |
Thank you for your answer, this code does not modify or overwrite the native Bangle.setHRMPower function. I’m very open to receiving other comments or suggestions! |
adding app.js and icone
metadata.js , ahrm.png
|
Is it possible to modify the hrm to use this? From what I can tell, this looks like it's another (maybe better) hrm algorithm, so it would make sense to have this modify the existing hrm... but correct me if I'm wrong about anything! Also, you don't need to add more changelog entries if it's all in the first pr. Since this will be the first pr update on this app, all the changes can be bundled into 0.01 :) |
|
Yes! the algorithm surely decides to switch on and off the hrm in a smarter way. I originally designed this app to function primarily as data logger (saving to CSV). The main goal was to move away from fixed-interval measurements and instead use a 'smart' approach—triggering measurements only when it is actually useful to have the heart rate data, i tested it for a long time while i was writing my thesis at university, and it worked !
The answer is, Yes it is possible to use it to override the original HRM, but after you should consider the differences from the default one. The logic is explained in the Readme file. Let me know your thoughts on this! Also, I’ve adjusted the changelogs as requested. I'll bundle all the changes into the initial 0.01 release." ;) |
bobrippling
left a comment
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.
Thanks for the app! A few comments
Co-authored-by: Rob Pilling <robpilling@gmail.com>
No description provided.