Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Incorrect unit of measurement for register 1052 in XLSX #30

Closed
sfstar opened this issue May 16, 2023 · 2 comments
Closed

Incorrect unit of measurement for register 1052 in XLSX #30

sfstar opened this issue May 16, 2023 · 2 comments

Comments

@sfstar
Copy link

sfstar commented May 16, 2023

Hello and thank you for opening the code to the modbus server component of the victron ecosystem.
There seems to be a mismatch between the values documented and actually returned for the dbus path: /ErrorCode

The XLSX states:
image

and attributes.csv contains:
'com.victronenergy.pvinverter,/Ac/Power,i,kW,1052,int32,1,W'

The value currently returned by 1052 is wattage based (as matches the attributes.csv file).
However, the XLSX states kW as the correct unit of measurement.
Is the XLSX incorrect or should the value returned be kW based but isn't encoded as such?

For reference the issue on my integrations issue tracker:
sfstar/hass-victron#13

@sfstar
Copy link
Author

sfstar commented Feb 27, 2024

@izak could you perhaps give me any insight as to how this discrepancy needs to be addressed/interpreted?

@izak
Copy link
Collaborator

izak commented Feb 27, 2024

This is already corrected in the xlsx. But thanks for reminding me that we need to update the one on the main website to match the 3.20 release.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants