adafruit / MAX31850_DallasTemp

A version of the DallasTemp Arduino library with MAX31850 support (Requires OneWire with MAX31850 support!)
14 stars 10 forks source link

sync with upstream #7

Open karlg100 opened 3 years ago

karlg100 commented 3 years ago

I just submitted a patch to the upstream Arduino-Temperature-Control-Library project to detect and support MAX31850, including fault detection and reporting.

https://github.com/milesburton/Arduino-Temperature-Control-Library/pull/183

I see this version also doesn't handle fault flags correctly, adversely affecting the precision of the readings.

ladyada commented 3 years ago

hi once that is merged we can sync w/upstream. please let us know when it is merged!

karlg100 commented 3 years ago

will do! Thanks!

karlg100 commented 3 years ago

One question / request - I don't have all the devices the upstream library supports. Would anyone at Adafruit be interested/able in testing the patch with all the other 1-wire devices?

ladyada commented 3 years ago

we would not be able to test the patch with other devices, we would only test the AMX31850