Different firmware different keys?

[readeral]

Hi Aaron,

Thanks for this project. I'm wondering if it's still working for you, or if there have been universal updates to the keys for all inverters via firmware.

For example, in your code the key for IP address is 6180_104A9A00 but on my recently installed inverter the IP address key is 6180_104AB700

Is it worth a pull request to update these keys, or is it potentially a case of 'different model, different keys'?

Al

[aaronferrucci]

Hey Al, thanks for your note. I capture from my inverter daily, and recent captures look fine (the IP address is reported correctly). My firmware version is 2.3.1.4 (or 2.3.1.R on the inverter web page); what firmware version are you seeing from your inverter? If I'm out of date I'm inclined to update it, and then a pull request would be great.

I'm assuming you saw the issue, then debugged it down to the changed IP address code. How did the bug present itself? Just a blank IP field in the report file, or something more unpleasant?

[readeral]

Ok I'm on totally different firmware (1.3.17.R) so that's likely the issue. Either a different inverter (Sunny boy 5.0, SB5.0-1AV-40 843) or might be a regional thing too. I'm in Australia.

The issue was a blank field, yeah, and so I went digging. Prior to finding your library I'd started reverse engineering the AngularJS app that runs from the inverter's webserver, so wasn't surprised to find a different key.

In any case, it's not a big deal - tbh I wrote the issue before realising it was the only different key in your library, expecting to find a number of them.

[aaronferrucci]

Ah, my inverter is a Sunny Boy 3.0 (SB3.0-1SP-US-40 816), in the US.

Hm, since it's just the one key - at least so far - I'm inclined to fix it with a simple hack: check both keys, and let either one set the "ip address" value:

      } else if (key2 === "6180_104A9A00") {
        transformedDevInfo["ip address"] = deeply_buried_value;
      } else if (key2 === "6180_104AB700") {
        transformedDevInfo["ip address"] = deeply_buried_value;
      }

This way you don't have to maintain a separate fork just for the other key, and I get the benefit of your testing on a different inverter. What do you think?