search

sparkfun / SparkFun_Bio_Sensor_Hub_Library

26 stars 33 forks source link

Add support for MAX30102 Sensor #12

Closed ChinmayPJE closed 3 years ago

ChinmayPJE commented 4 years ago

Add support for MAX30102 Sensor

ChrisIdema commented 3 years ago

I have the same issue. It reads raw led values just fine. It detects the finger. But heart rate only reads once in a while.

edspark commented 3 years ago

Sorry for the late reply, let me see what that would entail.

ChrisIdema commented 3 years ago

I've created a pull request. I've fixed some bugs to get it to work on the ESP32 and I fixed a bug in the delay of the readFillArray so that function is no longer too slow. See my pull request.

I've got it to work with the MAX30101 and the MAX30102 without making any further modifications! So my issues weren't related to the MAX30102 after all.

The problem I experienced was that the MAX32664 fills its FIFO very quickly. You have to continuously check the FIFO or else it will fill up and first the values will lag behind and then you will get invalid values because there are FIFO errors. I haven't found a way to avoid this without reading very quickly. I've created an example that reads it quick enough: see my pull request.

edspark commented 3 years ago

Thanks @ChrisIdema I'll make comments on your pull request as it's a little out of the scope of this issue. @ChinmayPJE this library does not directly support the MAX30101 or the MAX30102, that's the job of the MAX32664 on the product. The MAX32664 has firmware which supports both the MAX30101 and MAX30102. You can see that in the datasheet under the second paragraph on the first page which references "Version A", the version on this product.