-
Notifications
You must be signed in to change notification settings - Fork 635
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
RF bridge code learn problem #344
Comments
The latest version in the dev branch enqueues the RF messages and sends them using a ticker so they don't interfere with other modules (like MQTT). Can you try it to see if it solves the issue? |
Original comment by skorc99 (Bitbucket: skorc99, GitHub: skorc99): Hello, now it is sending RF codes and also learn is working. One issue still remains- after MQTT and DOMOTICZ are enabled it is not possible to save changes - i try to add IDX number but it is not saved. I will reflash and first add all IDX, RF codes and after that is saved i will try to enable MQTT ... U |
I am not experiencing this. With MQTT connected and Domoticz enabled I can save IDX and after reboot they are still there... Serial debug is disabled for RFBridge but you can still connect to it via telnet and read the messages. Maybe you can copy here part of the debug log (saving changes from web ui, turning on/off a switch,...) |
Original comment by Iván Conde (Bitbucket: ciconde, GitHub: Unknown): Similar here with a Sonoff S20 and latest firware and mosquitto broker. Everything is working fine until I enable MQTT. By the way and thus any changes not saved after enabling MQTT and getting resetted are lost. Even more, as webpage is on explorer's cache, after reset it seems operative, because websocket fail to send config doesn't cause any visible error. I think the webpage should warn when unable to communicate with device, but that is another topic. Debug by telnet (no serial available right now)
Unid did a reset cycle after that, and then died until hard reset. |
Original comment by skorc99 (Bitbucket: skorc99, GitHub: skorc99): My debug-all ok until MQTT enabled. After that i could not save changes and no output with telnet...
|
Issue #287 was marked as a duplicate of this issue. |
Original comment by Iván Conde (Bitbucket: ciconde, GitHub: Unknown): I must confirm it is still happening on 1.11.0, for the note |
Issue #314 was marked as a duplicate of this issue. |
Apparently it's a problem related only to MQTT, not the rfbridge. Can you test with another board (like a normal wemos or nodemcu)? Does it connect to your MQTT broker? |
It's an MQTT issue then. Tasmota uses a different MQTT client so it could be the client or the implementation. Best way to debug would be to decaod the error stack trace if you know how to do it. You could also remove things from the equation, like disable Domoticz to see if the error is in the Domoticz code or in the main MQTT code. You can test the same client Tasmota uses by setting MQTT_USE_ASYNC to 0 in the general.h file. |
Original comment by skorc99 (Bitbucket: skorc99, GitHub: skorc99): MQTT_USE_ASYNC to 0 now it is stable but i can only control from web.. I can also now confirm that web is working in safari. I tested:
So it is still MQTT issue.. U |
I have found an overflow bug in the mqtt code when the received payload is bigger than 255 chars, which happens with domoticz messages often. I have pushed the fix to the dev branch (fd02236). Can you please test if this is the issue you are facing? |
Original comment by Iván Conde (Bitbucket: ciconde, GitHub: Unknown): Definitely. Domoticz mqtt disabled, ESPurna stable. What surprises me a lot is the fact that i have the domoticz integration disabled O_o Anyway, I think it's confirmed that receiving domoticz mqtt messages causes the issue. |
Yeah, that's because the firmware still subscribes to the domoticz out topic even if domoticz is not enabled but it does not process the messages. Unfortunately, another module was choking with the messages. I have changed that behavior too with the latest commit (c0d63ff). Now it only subscribes to the domoticz out topic if domoticz support is enabled (and unsubscribes if the user disables it). |
Original comment by Iván Conde (Bitbucket: ciconde, GitHub: Unknown): Nice commit, improves bandwidth efficiency. That choking module... Guess we should escalate the issue. Will you open this issue? |
Released with 1.11.2 |
Removing milestone: 1.11.2 (automated comment) |
Original comment by Iván Conde (Bitbucket: ciconde, GitHub: Unknown): Thank you very much, Xose! |
Originally reported by: skorc99 (Bitbucket: skorc99, GitHub: skorc99)
Hello,#
i tested again new version HW different way..
1.) erased flash -OK
2.) OFF rf
3.) flashed latest version
) MQTT off
DOMOTICZ off
5.) Learned CODES - all OK
rebooted
All codes stored... after reboot...
6.) MQTT ON
DOMOTICZ ON
saved...
after step 6 RF bridge stopped sending RF codes and i can not turn off MQTT ...
I think MQTT is the issue...
U
The text was updated successfully, but these errors were encountered: