letscontrolit / ESPEasy

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

[ESPEasy p2p] C013_SensorInfoStruct does not include plugin specific settings #3264

Open TD-er opened 4 years ago

TD-er commented 4 years ago

These 3 plugins will very likely initialize the remote plugin with wrong value names and types:

These need some extra information like the type of output values and their names Also the number of output values can change among these plugins.

See usage of C013_SensorInfoStruct

Related: #3105

PR #3265 must be merged first as it also prepares fixing this.

C013_SensorInfoStruct must then be extended with plugin output type and task variable selection for the shared task. Must also check if older builds of ESPEasy will allow the extended struct.

N.B. also add a version nr to the struct, to overcome newer additions or maybe transistions of perhaps check to see if a receiving node only allows certain versions of the struct to be received.

tonhuisman commented 1 year ago

Not sure if this is still relevant, because of the age of the issue...

TD-er commented 1 year ago

Yep, still kinda relevant. I have made some changes recently to at least sync some parameters like nr of values, but it still is not complete.