-
-
Notifications
You must be signed in to change notification settings - Fork 97
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
[Feature Request]: Option to differenciate generated sensor names #21
Comments
The entity_id's are generated from the user-readable names. What I can do is to more dynamically generate those names Proposal:
What I won't do is add extra configuration parameters like a "custom prefix" in the config flow. Such parameters are always rejected by the HA Core team (same thing for allowing a customized update interval, cfr. discussion for Goodwe integration) |
I see... I am so used to HACS integrations that I am not really used to HA standards :) . Well my use case is not a common one as I have two isolated inverters managed in my home assistant (they are from different houses). In other integrations I have like the shelly relays I can rename them in the device so they are already imported with a different name but I understand that this is not really possible here and duplication is unavoidable. I like your proposal and think that would help users. At least making something different for the power meter would help to more recent users that already have trouble with the concept of input power vs active power vs power meter active power. Thanks for your time answering, you are doing a really great work with the integration (and with the register write feature). I am currently figuring out how to connect a modbus-rtu to modbus-tcp device to the inverter to avoid the unstable one they have for the app... that way write permission would be out of the box without authentication I guess |
Totally off topic but finally I was able to connect to the inverter using modbus tcp without using the fusion solar port provided by huawei. I used an Elfin-EW11 device to connect to the inverter using modbus-rtu and forwarding it to a modbus-tcp. Did not tried writing registers yet but seems to be an easy alternative to connecting to internal wifi once port 502 is closed in latests firmwares |
Can you let me know how you connected your rtu-to-tcp dongle? Which pins did you connect? |
Ok, so you aren't using the GND/EN+ (12V) pins, that was what I was wondering about. I guess you are talking about Abnormal status of power meter after upgrade to SPC117 firmware version on SUN2000 (2-6)KTL-L1 inverters ? I already checked my communication cable, and on 485B2/485A2 (ports 3/4) it has the wires to my LUNA2000 battery attached, while the 485B1/485A1 (ports 1/2) have the wires from the power meter attached. I just had to do the upgrade to SPC117, and indeed: the power meter values are frozen now. I'll have to rewire the communication cable to get it working again. |
Using gnd at least seems to be a good practice but as I had a 5v source near the device (for an ESP32 controlling the garage door) and the meter did not use it then decided to try that way... will think about it for the second inverter. However I don't really knew about that 12v pin you are talking about (which one is it?) EDIT: I found it: pin 6... the diagram I had showed 5 and 6 as GND...probably from another version... I wish it was that post as the ones I read did not specified how to do the re-discover process and had to figure out that part by myself |
Did a test with a multimeter and was not able to get a 12V reading between pins 6-7 |
Describe your feature request
When you configure one inverter you get two active power sensors. Home assistant renames one to active_power and the other one to active_power_1. This may be confusing at first to differenciate without looking at the parent entity but you can handle it without problems.
When you have two inverters you get also active_power_2 and active_power_3 and all the other sensors to _2 which refer to one or other inverter depending on the order you added them. This is a bit more problematic to manage in the long term remembering which one is suffixed with _1.
Using an automated prefix would break backwards compatibility and it is also hard to use a good one as inverter model can be also duplicated and serial number is not very human friendly.
I would like better to have an optional alias parameter to label the inverter's generated sensors with a custom prefix you can remember when using the sensors in cards/automations and so on
NOTE: I am waiting for a more final version to migrate and only temporary tested your login and write branch so I apologize if this do not applies to the main branch
The text was updated successfully, but these errors were encountered: