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

Fix IR and RED values interpretation from Arduino #3

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

RVillani
Copy link

@RVillani RVillani commented Jun 6, 2017

Hi, folks!

I'm using your shield and scripts for a medical application. Thanks a lot for them!

I've fixed a bug on the final SpO2 value displayed on the Processing software.

IR and RED values are sent from Arduino in that order and were being read as RED and IR by the Processing script. That changed the final SpO2 value to about 75% in a healthy person, whereas the standard is about 95%. It was also inverting the variation on the oxygen levels, eg., by holding your breath the value was raising instead of lowering as it should. Now it varies as expected.

@VenkateshBhat
Copy link
Contributor

Hello Rodrigo,
Thanks for your contribution, we truly appreciate your time and effort to make the algorithm better. We will review the code and merge it.
Thanks again,

@RVillani
Copy link
Author

RVillani commented Jun 7, 2017

Thank YOU guys for making it open source ^_^

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