eclipse-archived / smarthome

Eclipse SmartHome™ project
https://www.eclipse.org/smarthome/
Eclipse Public License 2.0
862 stars 787 forks source link

[OneWire] simplify multisensor configuration and improve thing discovery #6835

Closed J-N-K closed 5 years ago

J-N-K commented 5 years ago

This PR improves the adnvanced (BMS/AMS) multisensor discovery as requested e.g. here. From now on you only have to set the sensor id of the main sensor, additional sensors are auto-discovered. Also includes code improvements for the discovery service which will make future additions a lot simpler.

Signed-off-by: Jan N. Klug jan.n.klug@rub.de

staehler commented 5 years ago

@J-N-K First of all, thank you for your work on the onewire binding. As my MS-TH isn't recognized correctly by the discovery, I did some Google research and maybe this is usefull for others. I didn't knowm how to change an ID of the sensor in /26.pages/page.3 Maybe someone want's to add this to the binding documentation (I'm no developer and not familiar with git, sorry): Step-by-step instructions:

  1. Login onto your owserver host
  2. check, which sensor belongs to family 26: owdir / | grep 26.
  3. check which type the sensor is: owread /26.<sensornumber>/MultiSensor/type
  4. find the correct identification (0x00 = generic/MS-T, 0x19 = MS-TH, 0x1A = MS-TV, 0x1B = MS-TL, 0x1C = MS-TC)
  5. check identification of page.3: owread --hex /26.<sensornumber>/pages/page.3
  6. change identification in page.3 to e.g. 19: owwrite --hex /26.<sensornumber>/pages/page.3 19
  7. check identification of page.3 again: owread --hex /26.<sensornumber>/pages/page.3

Now the autodiscovery should work correct.

Thanks.

openhab-bot commented 5 years ago

This pull request has been mentioned on openHAB Community. There might be relevant details there:

https://community.openhab.org/t/humidity-channel-missing-in-latest-onewire-binding/63840/24