SML Script with multiple Meters get's wrong assigment

kadrim commented 5 months ago

PROBLEM DESCRIPTION

Having multiple Meters via SML is supported but some fields/registers are wrongly assigned. For example: define 2 Meters for SML

Maybe just ISKRA related?

REQUESTED INFORMATION

Make sure your have performed every step and checked the applicable boxes before submitting your issue. Thank you!

[x] Read the Contributing Guide and Policy and the Code of Conduct
[x] Searched the problem in issues
[x] Searched the problem in discussions
[x] Searched the problem in the docs
[x] Searched the problem in the chat
[x] Device used (e.g., Sonoff Basic): Wemos D1 mini
[x] Tasmota binary firmware version number used: 13.4.0
- [ ] Pre-compiled
- [x] Self-compiled
[x] Flashing tools used: platformIO
[x] Provide the output of command: Backlog Template; Module; GPIO 255:
```
Configuration output here:
```

10:13:49.875 RSL: RESULT = {"NAME":"Generic","GPIO":[1,1,1,1,1,1,1,1,1,1,1,1,1,1],"FLAG":0,"BASE":18} 10:13:50.083 RSL: RESULT = {"Module":{"1":"Sonoff Basic"}} 10:13:50.338 RSL: RESULT = {"GPIO0":{"32":"Button1"},"GPIO1":{"0":"None"},"GPIO2":{"0":"None"},"GPIO3":{"0":"None"},"GPIO4":{"0":"None"},"GPIO5":{"0":"None"},"GPIO9":{"0":"None"},"GPIO10":{"0":"None"},"GPIO12":{"224":"Relay1"},"GPIO13":{"320":"Led_i1"},"GPIO14":{"0":"None"},"GPIO15":{"0":"None"},"GPIO16":{"0":"None"},"GPIO17":{"0":"None"}}

- [x] If using rules, provide the output of this command: `Backlog Rule1; Rule2; Rule3`:
```lua
  Rules output here:

[x] Provide the output of this command: Status 0:


STATUS 0 output here:
10:14:55.390 RSL: STATUS = {"Status":{"Module":1,"DeviceName":"Tasmota","FriendlyName":["Tasmota"],"Topic":"tasmota_1B0FE3","ButtonTopic":"0","Power":0,"PowerOnState":3,"LedState":1,"LedMask":"FFFF","SaveData":1,"SaveState":1,"SwitchTopic":"0","SwitchMode":[0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0],"ButtonRetain":0,"SwitchRetain":0,"SensorRetain":0,"PowerRetain":0,"InfoRetain":0,"StateRetain":0,"StatusRetain":0}}
10:14:55.396 RSL: STATUS1 = {"StatusPRM":{"Baudrate":9600,"SerialConfig":"8N1","GroupTopic":"tasmotas","OtaUrl":"http://ota.tasmota.com/tasmota/release/tasmota-4M.bin.gz","RestartReason":"Software/System restart","Uptime":"0T00:17:34","StartupUTC":"2024-03-25T08:57:21","Sleep":50,"CfgHolder":4617,"BootCount":2,"BCResetTime":"2024-03-25T09:57:28","SaveCount":9,"SaveAddress":"FB000"}}
10:14:55.400 RSL: STATUS2 = {"StatusFWR":{"Version":"13.4.0(iskra_mt691)","BuildDateTime":"2024-03-25T09:44:35","Boot":31,"Core":"2_7_6","SDK":"2.2.2-dev(38a443e)","CpuFrequency":80,"Hardware":"ESP8266EX","CR":"388/699"}}
10:14:55.405 RSL: STATUS3 = {"StatusLOG":{"SerialLog":0,"WebLog":2,"MqttLog":0,"SysLog":0,"LogHost":"","LogPort":514,"SSId":["***",""],"TelePeriod":300,"Resolution":"558180C0","SetOption":["00008009","2805C80001000600003C5A0A192800000000","00000080","00006000","00004000","00000000"]}}
10:14:55.417 RSL: STATUS4 = {"StatusMEM":{"ProgramSize":772,"Free":1272,"Heap":10,"ProgramFlashSize":4096,"FlashSize":4096,"FlashChipId":"16405E","FlashFrequency":40,"FlashMode":"DOUT","Features":["0809","879AC78F","043E8001","000000CF","010013C0","C000F989","00004084","00201000","54000020","00000080","00000000"],"Drivers":"1,2,!3,!4,!5,!6,7,!8,9,10,12,!16,!18,!19,!20,!21,!22,!24,26,!27,29,!30,!35,!37,38,!45,50,62,!68","Sensors":"1,2,3,4,5,6,53","I2CDriver":"7"}}
10:14:55.423 RSL: STATUS5 = {"StatusNET":{"Hostname":"tasmota-1B0F****","IPAddress":"192.****","Gateway":"192.***","Subnetmask":"255.***","DNSServer1":"192.***","DNSServer2":"0.0.0.0","Mac":"****","Webserver":2,"HTTP_API":1,"WifiConfig":4,"WifiPower":17.0}}
10:14:55.426 RSL: STATUS6 = {"StatusMQT":{"MqttHost":"","MqttPort":1883,"MqttClientMask":"DVES_%06X","MqttClient":"DVES_1B0FE3","MqttUser":"DVES_USER","MqttCount":0,"MAX_PACKET_SIZE":1200,"KEEPALIVE":30,"SOCKET_TIMEOUT":4}}
10:14:55.431 RSL: STATUS7 = {"StatusTIM":{"UTC":"2024-03-25T09:14:55Z","Local":"2024-03-25T10:14:55","StartDST":"2024-03-31T02:00:00","EndDST":"2024-10-27T03:00:00","Timezone":"+01:00","Sunrise":"06:41","Sunset":"19:10"}}
10:14:55.437 RSL: STATUS10 = {"StatusSNS":{"Time":"2024-03-25T10:14:55","Z1":{"Total_in":0.000,"Power_cur1":0,"Total_out":0.000,"Meter_id":"","Power_cur2":0},"Z2":{"Total_out":0.000,"Meter_id":""}}}
10:14:55.443 RSL: STATUS11 = {"StatusSTS":{"Time":"2024-03-25T10:14:55","Uptime":"0T00:17:34","UptimeSec":1054,"Heap":10,"SleepMode":"Dynamic","Sleep":50,"LoadAvg":19,"MqttCount":0,"POWER":"OFF","Wifi":{"AP":1,"SSId":"***","BSSId":"***","Channel":2,"Mode":"11n","RSSI":68,"Signal":-66,"LinkCount":1,"Downtime":"0T00:00:06"}}}

- [ ] Set `weblog` to 4 and then, when you experience your issue, provide the output of the Console log:
```lua
  Console output here:

TO REPRODUCE

Compile a Firmware with SML and SCRIPS enabled, i.e.:

platformio_tasmota_cenv.ini

[env:tasmota-d1mini-iskra-mt691]
extends                    = env:tasmota-4M
build_flags                = ${env:tasmota-4M.build_flags} -DFIRMWARE_ISKRA_MT691

user_config_override.h

#ifdef FIRMWARE_ISKRA_MT691

    #undef CODE_IMAGE_STR
    #define CODE_IMAGE_STR "iskra_mt691"

    // Put here your override for firmware tasmota32-iskra_mt691
    #ifndef USE_SCRIPT
    #define USE_SCRIPT
    #endif
    #ifndef USE_SML_M
    #define USE_SML_M
    #endif
    #ifdef USE_RULES
    #undef USE_RULES
    #endif

#endif

Upload the Firmware
Finish the usual Setup (Wifi etc.)
Enable Scripts and add this one (taken from Documenation https://tasmota.github.io/docs/Smart-Meter-Interface/), adjusted to fit two separate Meters:

>D
>B
=>sensor53 r
>M 2
+1,3,s,0,9600,Z1
1,77070100010800ff@1000,Total Consumed,kWh,Total_in,3
1,77070100100700ff@1,Current Consumption,W,Power_cur1,0
1,77070100020800ff@1000,Total Delivered,kWh,Total_out,3
1,77070100000009ff@#,Service ID,,Meter_id,0|
+2,14,s,0,9600,Z2
2,77070100100700ff@1,Current Production,W,Power_cur2,0
2,77070100020800ff@1000,Total Delivered,kWh,Total_out,3
2,77070100000009ff@#,Service ID,,Meter_id,0|
#

you will see a Page like this:

EXPECTED BEHAVIOUR

The Current Production should be assigned to Z2 but instead it is assigned to Z1. See the line 2,77070100100700ff@1,Current Production,W,Power_cur2,0 which should assign it to the second Meter

SCREENSHOTS

see above

ADDITIONAL CONTEXT

This issue can be tested without any real hardware other than an ESP.

(Please, remember to close the issue when the problem has been addressed)

gemu2015 commented 5 months ago

your script contains illegal characters at the end of 2 lines ("|") remove them and it will work

gemu2015 commented 5 months ago

i see this error comes from the example. will fix it.

kadrim commented 5 months ago

OMG, stupid me - thanks alot @gemu2015 !

closing this right now

arendst / Tasmota