search

adafruit / Adafruit_BMP280_Library

Arduino Library for BMP280 sensors
253 stars 186 forks source link

Reworked for combined T-P burst reads (see DS 3.10). #7

Closed mmehr2 closed 7 years ago

mmehr2 commented 7 years ago

This deals with the same burst read issue that I reworked in the BME280 library code (PR#14). It only affects readPressure() so that it can read the temperature used for compensation from the same sample as the pressure. Added readAll() to get both T and P from same sample.

mmehr2 commented 7 years ago

See my comments in the similar closed PR for the BME library. I discovered that I got the order of registers wrong for the new read functions, so this code doesn't work. I will resubmit once I have tested it on a real Arduino or two, and with better test coverage.

Newbie mistakes and just not getting proper tests done. I am abashed. But will try again later, with the real deal. I will still only have minimal testing (hopefully one Arduino family board, maybe a WICED Feather from Adafruit as well). Look for the resubmission in a few weeks. My sincere apologies for the mistake.