hixfield/hixarduino

Different classes to facilitate writing Arduino and ESP (8266) based projects

#HixArduino Objected oriented components for working with the Arduino:

• HixPin
• HixPinDigital
• HixPinDigitalOutput
• HixPinDigitalInput
• HixPinPWM
• HixPinPWM28KHz
• HixPinAnalogInput
• HixTimeout
• HixDS18B20
• HixAnalogJoystick

##HixPin

hixarduino's Issues

HixDS18B20 class with many Dallas termometers

Hi @hixfield and thanks for testing my lib on ESP8266 platform. I hope it works fine. Did you have troubles with OneWire lib that you decided to use mine?

I see one issue with using HixDS18B20 with more than 1 thermometer - it will simply not work if more than 1 therm will be connected to the bus.

1. HixDS18B20::begin() will return false (error), since OneWireNg::search() will return OneWireNg::EC_MORE in this case:

  ec = m_pOneWire->search(m_sensorId);
  return ec == OneWireNg::EC_DONE;

Of course this shouldn't be treated as an error. It simply informs the caller - more devices are available to fetch, call me more to get their ids...

2. Why constraint to only single connected therm? Don't you think this simplification goes too far. You may have implemented search routine returning all Dallas therms supported by your lib (using family code filtering) and then use getTemp() to fetch temperature for a given id. There is also possible to send CONVERT_T to all devs connected via OneWireNg::addressAll() and next read the result on-by-one. This dramatically reduces the read operation time and gives temperature at a given moment of time measured at all devices simultaneously.

Thanks,
Piotrek