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

MQTT truncated with multiple DS18B20 sensors #114

Closed
ramsaymike opened this issue Feb 26, 2017 · 12 comments
Closed

MQTT truncated with multiple DS18B20 sensors #114

ramsaymike opened this issue Feb 26, 2017 · 12 comments
Labels
help needed Action - Asking for help from the community

Comments

@ramsaymike
Copy link

Trying to use multiple DS18B20 sensors with Sonoff Basic It seems that the MQTT message is truncated at 239 characters for some reason - part way through sensor 3 address. This is seen in both the console and Mosquitto. The web interface reports all data (well, up to the 4 I tried anyway) as expected. Any ideas?

@lucwuyts
Copy link

maybe you need to increment the MQTT_MAX_PACKET_SIZE

@arendst
Copy link
Owner

arendst commented Feb 26, 2017

Change in sonoff.ino line 125 MESSZ to a higher value. You'll then also have to increase MQTT_MAX_PACKET_SIZE.

@arendst arendst added the help needed Action - Asking for help from the community label Feb 26, 2017
@ramsaymike
Copy link
Author

Thanks - MESSZ was the problem! And it was sitting right there in the comment!!!
BTW, this stuff is a work of art - fantastic job. I just wish I could use it for all my 8266s.

@2delarosa
Copy link

@ramsaymike, I'm looking to attach DS18B20 Sensors to either the Sonoff Basic or POW and monitor the temperature of my aquarium. How are you physically attaching the DS18B20?

@ramsaymike
Copy link
Author

ramsaymike commented Feb 27, 2017 via email

@2delarosa
Copy link

Thanks. Any need for calibration from the software side?

@ramsaymike
Copy link
Author

ramsaymike commented Feb 27, 2017 via email

@altelch
Copy link
Contributor

altelch commented Feb 27, 2017

The DS18x20 is factory calibrated but readings differ about +/- 0.5°C.

@2delarosa
Copy link

Thanks. I ordered several sensors to try them out.

arendst added a commit that referenced this issue Mar 3, 2017
4.0.0 20170303
* Add define to remove config migration code for versions below 3.0 (See
Wiki-Upgrade-Migration path)
* Free memory by switching from String to char[]
* Raised Sonoff Led PWM frequency from 200Hz to 432Hz in search of
stability (hardware watchdog timeouts) (#122)
* Increase message size and suggested minimum MQTT_MAX_PACKET_SIZE to
512 (#114, #124)
* Remove runtime warning message regarding MQTT_MAX_PACKET_SIZE too
small as it is now moved to compile time (#124)
* Fix possible panics with web console and http commands while UDP
syslog is active (#127)
* Add optional static IP address (#129)
* Add define ENERGY_RESOLUTION in user_config.h to allow user control
over precision (#136)
@ramsaymike
Copy link
Author

ramsaymike commented Mar 22, 2017 via email

@davidelang
Copy link
Collaborator

davidelang commented Mar 22, 2017 via email

@ramsaymike
Copy link
Author

ramsaymike commented Mar 22, 2017 via email

curzon01 pushed a commit to curzon01/Tasmota that referenced this issue Sep 6, 2018
4.0.0 20170303
* Add define to remove config migration code for versions below 3.0 (See
Wiki-Upgrade-Migration path)
* Free memory by switching from String to char[]
* Raised Sonoff Led PWM frequency from 200Hz to 432Hz in search of
stability (hardware watchdog timeouts) (arendst#122)
* Increase message size and suggested minimum MQTT_MAX_PACKET_SIZE to
512 (arendst#114, arendst#124)
* Remove runtime warning message regarding MQTT_MAX_PACKET_SIZE too
small as it is now moved to compile time (arendst#124)
* Fix possible panics with web console and http commands while UDP
syslog is active (arendst#127)
* Add optional static IP address (arendst#129)
* Add define ENERGY_RESOLUTION in user_config.h to allow user control
over precision (arendst#136)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help needed Action - Asking for help from the community
Projects
None yet
Development

No branches or pull requests

6 participants