Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

4 dallas sensors - MQTT String corrupt #637

Closed
Schnittcher opened this issue Jul 24, 2017 · 4 comments
Closed

4 dallas sensors - MQTT String corrupt #637

Schnittcher opened this issue Jul 24, 2017 · 4 comments
Labels
enhancement Type - Enhancement that will be worked on

Comments

@Schnittcher
Copy link

Schnittcher commented Jul 24, 2017

Hello,

on the Sonoff Switch, we have 4 dalls sensors.
MQTT sends a corrupted string:

{"Time":"2017-07-24T12:23:35", "DS18x20":{"DS1":{"Type":"DS18S20", "Address":"10D4B431030800E6", "Temperature":19.0}, "DS2":{"Type":"DS18S20", "Address":"105A7F30030800C2", "Temperature":19.8}, "DS3":{"Type":"DS18S20", "Address":"10BB863003080089", "Temperature":19.2}, "DS4":{"Type":"DS18S20", "Address":"10C72D3203080028", "Temperature":19.8}}, "TempUnit":"

When we have 2 sensors on the switch, the MQTT string is fine:

{"Time":"2017-07-24T12:24:52", "DS18x20":{"DS1":{"Type":"DS18S20", "Address":"10D4B431030800E6", "Temperature":19.1}, "DS2":{"Type":"DS18S20", "Address":"10C72D3203080028", "Temperature":19.8}}, "TempUnit":"C"}

Can anyone help?
Sorry for my bad english.

@davidelang
Copy link
Collaborator

davidelang commented Jul 24, 2017 via email

@Schnittcher
Copy link
Author

Thanks, I will try it.
The max message size is MESSZ, right?
Can I change this with a command? Or I need to flash it, again?

@davidelang
Copy link
Collaborator

davidelang commented Jul 24, 2017 via email

@arendst
Copy link
Owner

arendst commented Jul 26, 2017

You could also use the pre-compiled binary sonoff-ds18x20.bin which allows for more sensors.

In the next version I'll update the default buffer length anyway with some 4 bytes as the problem is introduced by adding the TempUnit field to the JSON message.

@arendst arendst added the enhancement Type - Enhancement that will be worked on label Jul 26, 2017
arendst added a commit that referenced this issue Jul 30, 2017
5.5.0 20170730
* Reduce code space by removing the following commands as they are
replaced by SetOption alternatives:
*   SaveState = SetOption0
*   ButtonRestrict = SetOption1
*   Units = SetOption2
*   MQTT = SetOption3
*   MQTTResponse = SetOption4
*   TempUnit = SetOption8
* Smoothing WS2812 animation poll, invert fade speed and max allowed
wakeup time down to 3000 seconds
* Fix initial button press detection
* Add support for Sonoff RF Bridge 433 using command RfKey
* Fix regression from 5.0.7 by increasing message buffer size from 360
to 368 to accomodate 4 x DS18x20 sensors (#637)
* Add GroupTopic to Topic test when using ButtonTopic/SwitchTopic to
send either ON/OFF or TOGGLE (#642)
* Adjust HLW calibration limits to accomodate HuaFan device and add
commands HlwPSet, HlwUSet and HlwISet (#654)
@arendst arendst closed this as completed Nov 7, 2017
curzon01 pushed a commit to curzon01/Tasmota that referenced this issue Sep 6, 2018
5.5.0 20170730
* Reduce code space by removing the following commands as they are
replaced by SetOption alternatives:
*   SaveState = SetOption0
*   ButtonRestrict = SetOption1
*   Units = SetOption2
*   MQTT = SetOption3
*   MQTTResponse = SetOption4
*   TempUnit = SetOption8
* Smoothing WS2812 animation poll, invert fade speed and max allowed
wakeup time down to 3000 seconds
* Fix initial button press detection
* Add support for Sonoff RF Bridge 433 using command RfKey
* Fix regression from 5.0.7 by increasing message buffer size from 360
to 368 to accomodate 4 x DS18x20 sensors (arendst#637)
* Add GroupTopic to Topic test when using ButtonTopic/SwitchTopic to
send either ON/OFF or TOGGLE (arendst#642)
* Adjust HLW calibration limits to accomodate HuaFan device and add
commands HlwPSet, HlwUSet and HlwISet (arendst#654)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Type - Enhancement that will be worked on
Projects
None yet
Development

No branches or pull requests

3 participants