search

letscontrolit / ESPEasy

Easy MultiSensor device based on ESP8266/ESP32
http://www.espeasy.com
Other
3.23k stars 2.2k forks source link

Problems with Nextion #2772

Closed schamu closed 4 years ago

schamu commented 4 years ago

I have an error with the Nextion plugin:   The command does not work in the rules: nextion, page99.t13.txt="%c_m2dhm%(% uptime%)"   The command works: nextion, page99.t14.txt="%ip%"   But it works in lines 1-10 for devices.   The following is displayed in the logfile: 1311958: EVENT: Rules#Timer = 7 1312012: ACT: nextion, page99.t13.txt = '0d00h22m' 1312020: ACT: nextion, page99.t14.txt = '192.168.178.14'

The following are also not sent to the Nextion: Nextion, page01.t10.txt="[BME280#Temperature]" Nextion, page01.t11.txt="[BME280#Humidity]" Nextion, page01.t12.txt="[BME280#Pressure]" Nextion, page05.t13.txt="[DS18b20#Temperature]"

When I send UPD commandos to the Nextion it works too. Only the rules do not work.

The error occurs at least since version mega-20191003

TD-er commented 4 years ago

See this issue describing what you probably experience: https://github.com/letscontrolit/ESPEasy/issues/2724

For example:

nextion, page99.t13.txt="% c_m2dhm%(% uptime%)"

Will then become:

nextion,'page99.t13.txt="%c_m2dhm%(%uptime%)"'

(added single quote to wrap the parameter and removed spaces after the %)

schamu commented 4 years ago

There is no space in my rules, it was a mistake in the post. I've already tried that with the quotes.

TD-er commented 4 years ago

The quotes wrapping the whole parameter?

schamu commented 4 years ago

yes

TD-er commented 4 years ago

Please post your relevant rules, which you have now active so I can see what is stored in your rules. Wrap them in 3x left quote (`) to make sure the syntax doesn't get mangled here in a comment.

schamu commented 4 years ago 
on Rules#Timer=7 do
 //Nextion,page99.t11.txt="0%sunrise% - %sunset%"
 nextion,'page99.t13.txt="%c_m2dhm%(%uptime%)"'
 nextion,page99.t14.txt="%ip%"
endon

on Rules#Timer=8 do
 Nextion,page01.t10.txt="[BME280#Temperature]"
 Nextion,page01.t11.txt="[BME280#Humidity]"
 Nextion,page01.t12.txt="[BME280#Pressure]"
 Nextion,page05.t13.txt="[DS18b20#Temperature]"
endon

on SendeSchalter do
 Nextion,page06.bt0.val=[schalter#sofa]
 Nextion,page06.bt1.val=[schalter#lichterkette]
 Nextion,page06.bt2.val=[schalter#terasse]
 Nextion,page06.bt3.val=[schalter#steckdose]
endon

only the %ip% is working

TD-er commented 4 years ago

Can you enable debug log and post the relevant log parts where the decoding of the rules are shown?

schamu commented 4 years ago

3723967: Command: event 3723967: EVENT: SendeSchalter 3724084: ACT : Nextion,page06.bt0.val=1 3724115: ACT : Nextion,page06.bt1.val=0 3724144: ACT : Nextion,page06.bt2.val=0 3724174: ACT : Nextion,page06.bt3.val=0 3728911: EVENT: Rules#Timer=7 3728962: ACT : Nextion,page99.t13.txt=''0d01h02m'' 3728967: ACT : Nextion,page99.t14.txt='192.168.178.14' 3732918: EVENT: Rules#Timer=8 3733004: ACT : Nextion,page01.t10.txt='20.7' 3733016: ACT : Nextion,page01.t11.txt='50.9' 3733029: ACT : Nextion,page01.t12.txt='984.6' 3733040: ACT : Nextion,page05.t13.txt='20.7'

TD-er commented 4 years ago

Is that with debug level enabled?

schamu commented 4 years ago

were can I enable the debug level?

TD-er commented 4 years ago

Tools => Advanced image

schamu commented 4 years ago

4623984: ACT : Event,SendeSchalter 4623992: Command: event 4623992: EVENT: SendeSchalter 4624089: ACT : Nextion,page06.bt0.val=0 4624121: ACT : Nextion,page06.bt1.val=0 4624150: ACT : Nextion,page06.bt2.val=0 4624180: ACT : Nextion,page06.bt3.val=0 4628913: EVENT: Rules#Timer=7 4628990: ACT : Nextion,page99.t13.txt=''0d01h17m'' 4628994: ACT : Nextion,page99.t14.txt='192.168.178.14' 4630934: DS: SP: 7b,1,ff,ff,7f,ff,ff,ff,ac,OK,8 4630934: DS : Temperature: 23.69 (28-61-64-12-29-d6-36-50) 4630936: EVENT: DS18b20#Temperature=20.89 4631125: SW : GPIO=12 State=1 Output value=1 4631129: EVENT: taste4#1=1.00 4631180: ACT : Publish /ESP_Display1/Bewegung,1 4631194: Command: publish 4631195: Publish: /ESP_Display1/Bewegung1 4632912: EVENT: Rules#Timer=8 4632997: ACT : Nextion,page01.t10.txt='20.9' 4633012: ACT : Nextion,page01.t11.txt='50.6' 4633024: ACT : Nextion,page01.t12.txt='984.2' 4633035: ACT : Nextion,page05.t13.txt='20.9'

TD-er commented 4 years ago

What build are you using? If that's with the Debug level set, then we're missing the actual debug log entries. It looks like BUILD_NO_DEBUG is defined, but that should only be for "minimal" or "normal" builds (or custom builds you made yourself)

And since the Nextion plugin is only present in "testing", there should be debug info.

schamu commented 4 years ago
Build:⋄ 20104 - Mega
System Libraries:⋄ ESP82xx Core 2_6_1, NONOS SDK 2.2.2-dev(38a443e), LWIP: 2.1.2 PUYA support
Git Build:⋄ mega-20191123
Plugins:⋄ 80 [Normal] [Testing] [Development]
Build Md5: 835d9c4739177bbad2821062ba8a35
Md5 check: passed.
Build Time:⋄ Nov 23 2019 03:12:49
Binary Filename:⋄ ESP_Easy_mega-20191123_dev_ESP8266_4M1M.bin
TD-er commented 4 years ago

I'm looking at the code of the Nextion plugin now. Apparently the command for the Nextion display is actually the name of the task. Is that true?

Also the Nextion plugin does not use the (new) standard parse functions, although I am not 100% sure that's what is causing the issue here.

schamu commented 4 years ago

In this build is all ok;

Build:⋄ 20103 - Mega
System Libraries:⋄ ESP82xx Core 2_5_2, NONOS SDK 2.2.1(cfd48f3), LWIP: 2.1.2 PUYA support
Git Build:⋄ mega-20190827
Plugins:⋄ 81 [Normal] [Testing] [Development]
Build Md5: 6dd455f8fbf8143bdd11bd953a4b2f
Md5 check: passed.
Build Time:⋄ Aug 27 2019 02:34:26
Binary Filename:⋄ ESP_Easy_mega-20190827_dev_ESP8266_4M.bin
TD-er commented 4 years ago

Can you test with this test build ?

schamu commented 4 years ago

This one? There ist the same error

Build:⋄ 20104 - Mega
System Libraries:⋄ ESP82xx Core 2.7.0-dev stage, NONOS SDK 2.2.2-dev(bb83b9b), LWIP: 2.1.2 PUYA support
Git Build:⋄ mega-20191123
Plugins:⋄ 79 [Normal] [Testing]
Build Md5: a667330ae76d2cfa961f72db502680
Md5 check: passed.
Build Time:⋄ Nov 23 2019 03:49:28
Binary Filename:⋄ ESP_Easy_mega-20191123_test_beta_ESP8266_4M1M.bin
schamu commented 4 years ago

Sorry, now the right one.

Looks good, I'll still test everything

schamu commented 4 years ago

It looks like everything works. Thank you for your good work

TD-er commented 4 years ago

OK, I will make it into a pull request and if someone else can test it, then it will be in the next nightly build.

thomastech commented 4 years ago

... and if someone else can test it, then it will be in the next nightly build.

Done! I confirmed that your test build restores the broken Nextion rules.

TD-er commented 4 years ago

While you're at it... I just finished changing other plugins too, which were probably effected in the same way. Test build is started, but if you don't mind, I will now try to get some sleep ;)

TD-er commented 4 years ago

The test build for this PR: ESPEasy_mega-20191123-3-PR_2774.zip

schamu commented 4 years ago

I tested this version on different controllers. I use the 8266_4M1M dev and normal versions with different sensors and switches / buttons and mqtt.

All functions work for me.

Thank you for your great work.