-
Notifications
You must be signed in to change notification settings - Fork 52
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
DTSD422-D3 - incorrect values returned #152
Comments
|
I fixed all, and now it is working like charm :) |
@mrlordziemniak What was the problem? Did you have to patch the code? Is there anything you can contribute back to the project? |
Hi kbialek, at first, thanks for your work. I'm able to read the data from my Solarman smart meter DTSD422-D3, but I don't know what exactly are the total energy data. I'm comparing these numbers with data from Solarman API, but the data from API are absolutely different. So what stands for Total Positive Energy or Total Positive Energy2, I mean it is daily, weekly, or all time? BTW: total active power and total active power 2 are correct, it's the same as from the API |
in k
because there is wrong modbus adress, look here https://github.com/mrlordziemniak/dtsd422_esphome/blob/main/dtsd422_esphome.yaml i have correct and working modbus adress for dtsd422-d3 but im using rs432 and esphome for this :) |
is there a way how to fix the [deye-inverter-mqtt] code? |
check this modbus adress https://github.com/mrlordziemniak/dtsd422_esphome/blob/main/dtsd422_modbus_register.md correct with this one from kbialek, there is problem onyl with S_WORD because solarman using strange sign interpretation, there is - when in first adddress is 8000H |
@Scorpion1211svk fyi, DTSD422 integration has been provided by the community. I do not own such a device, and thus cannot verify if the integration works correctly. However, with the input provided by @mrlordziemniak I'm now able to double check the setup.
I don't know that. Please check in the meter manual.
This is called Signed Magnitude representation and deye-inverter-mqtt is currently using it for a couple of registers in this meter integration, namely:
@mrlordziemniak In your esphome config, you apply lambda transformations to handle the sign properly. You do this for ActivePower 1-3, but not for ActivePower 4-6. Is there a reason for that? What about current readings, aren't they also signed, to represent energy import/export? |
Because from 1-3 there is active power with plus or minus, on 4-6 with my setup of ct clamp there is always plus sign so I read only data value without sign :) same with current 1-3 with sign and 4-5 always positive, I read data from sign register and from value register then I use lambda to compare sign, I don't know how do this in different simple way, I mean read double register compare with lambda and get correct reading because S_DWORD doesn't work for me :/ |
Thanks for the info. So you made it working with your setup 👍 Good to know that. Yeah, I think that esphome modbus component does not support signed magnitude representation, so your approach with the lambda is a valid workaround to the problem. |
I can provide full modbus register for all data for that meter, I have xls from my friend :) |
That would be great |
When do you expect to update the code? Just asking, ;) |
I'm waiting for the xls promised by @mrlordziemniak 🙂 |
How can i send you that file? |
Options:
|
Beta build is ready for testing Update list of MQTT topics is available here https://github.com/kbialek/deye-inverter-mqtt/blob/release/2024.04.1/docs/metric_group_igen_dtsd422.md |
Two things:
|
Thank you very, very much, it works perfectly now. |
Sounds great! In return I kindly encourage you to support my work and to buy me a coffee 😄 |
and on all Total Positive Energy CT1 CT2 etc is not correct value, any ideas why?
PL:
Udało się zrobić odczyt z tego licznika, ale nie wiem dlaczego wartości prądu jak są z minusem to dalej pokazuje jak z plusem, tj nie pojawia się minus, oraz pod rejestrami od total positive energy są jakieś głupoty
Originally posted by @mrlordziemniak in #41 (comment)
The text was updated successfully, but these errors were encountered: