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

SetOption136 disable TuyaSNS immediate publish #15216

Merged
merged 1 commit into from
Mar 25, 2022

Conversation

barbudor
Copy link
Contributor

Description:

Related issue (if applicable): fixes #14842

Summary

SetOption136 0 : default (legacy behavior) : publish an immediate tele/../SENSOR TuyaSNS message at each reception of individual value
SetOption136 1: disable this behavior while using immediate receive in Rules

Type of MQTT "spamming" with legacy configuration that is disabled with SO136 1

23:48:13.606 MQT: tele/nodemcu/SENSOR = {"TuyaSNS":{"Humidity":1.1}}
23:48:13.664 MQT: tele/nodemcu/SENSOR = {"TuyaSNS":{"CarbonDioxide":295}}
23:48:13.696 MQT: tele/nodemcu/SENSOR = {"TuyaSNS":{"TVOC":40}}

Non-breaking PR as the legacy behavior remains.

Tested on ESP8266 nodemcu using Tuya Simulator (cool product once you found when to switch it in english)
esp32-c3 to be tested by @loca5790

Checklist:

  • The pull request is done against the latest development branch
  • Only relevant files were touched
  • Only one feature/fix was added per PR and the code change compiles without warnings
  • The code change is tested and works with Tasmota core ESP8266 V.2.7.4.9
  • The code change is tested and works with Tasmota core ESP32 V.2.0.2.3
  • I accept the CLA.

NOTE: The code change must pass CI tests. Your PR cannot be merged unless tests pass

@barbudor
Copy link
Contributor Author

Also tadding @FrenkK

@barbudor
Copy link
Contributor Author

barbudor commented Mar 25, 2022

@arendst
please hold I left a debug log always on
Will fix this morning

It's ok, the logs are not in the commit

@arendst arendst merged commit 5494d47 into arendst:development Mar 25, 2022
@loca5790
Copy link

@barbudor tested on ESP32C3 working

@barbudor barbudor deleted the tuyasns_so_immediate_publish branch March 25, 2022 15:01
arendst added a commit that referenced this pull request Mar 25, 2022
Add command ``SetOption136 1`` to disable single sensor reports from Tuya devices while keeping teleperiod reports (#15216)
@loca5790
Copy link

loca5790 commented Mar 26, 2022

@barbudor excuse the ignorance but where can I access the latest merged binaries for the ESP32?

https://ota.tasmota.com/tasmota/ the 4m.bin?

@barbudor
Copy link
Contributor Author

Yes, once a PR has been merged and is part of Tasmota code, you will get the dev binaries automatically buit within a couple of hours in the OTA server you pointed

@loca5790
Copy link

Just checking thanks, couldn't find one that matched ESP32C3 just wanted to make sure.

@barbudor
Copy link
Contributor Author

@loca5790
Copy link

@barbudor thanks.

Working with logging suppressed like intended. Still getting continuous pings from the wifi chip cyclically at rate of around 1 ping per second. Appears to be linked to the Tuya MCU usage, will investigate more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants