Rename photovoltaic entities to "PV" to differentiate from thermal Solar

[NoRi2909]

The problem

In the Vicare integration there are seven entities which relate to PV installations, obvious from their entity IDs:

core/homeassistant/components/vicare/strings.json

Lines 349 to 372 in 1f9c06e

	"photovoltaic_power_production_current": {
	"name": "Solar power"
	},
	"photovoltaic_energy_production_today": {
	"name": "Solar energy production today"
	},
	"photovoltaic_energy_production_this_week": {
	"name": "Solar energy production this week"
	},
	"photovoltaic_energy_production_this_month": {
	"name": "Solar energy production this month"
	},
	"photovoltaic_energy_production_this_year": {
	"name": "Solar energy production this year"
	},
	"photovoltaic_energy_production_total": {
	"name": "Solar energy production total"
	},
	"photovoltaic_status": {
	"name": "Solar state",
	"state": {
	"ready": "Standby",
	"production": "Producing"
	}

These should have their entity names changed from "Solar …" to "PV …" to differentiate them from this group:

core/homeassistant/components/vicare/strings.json

Lines 293 to 303 in 1f9c06e

	"solar_power_production_today": {
	"name": "Solar energy production today"
	},
	"solar_power_production_this_week": {
	"name": "Solar energy production this week"
	},
	"solar_power_production_this_month": {
	"name": "Solar energy production this month"
	},
	"solar_power_production_this_year": {
	"name": "Solar energy production this year"

Here the use of "power" in the entity IDs is wrong and should be replaced with "energy".

Alongside the name of this entity

core/homeassistant/components/vicare/strings.json

Lines 33 to 35 in c294130

	"circulation_pump": {
	"name": "Circulation pump"
	},

should be changed to "Heating circulation pump" to make it consistent with this one:

core/homeassistant/components/vicare/strings.json

Lines 51 to 53 in c294130

	"domestic_hot_water_circulation_pump": {
	"name": "DHW circulation pump"
	},

Avoids major misunderstandings as in Germany the circulation pump is only known for DHW.

What version of Home Assistant Core has the issue?

2024.10.3

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

Viessmann ViCare

Link to integration documentation on our website

https://www.home-assistant.io/integrations/vicare/

[home-assistant]

Hey there @CFenner, mind taking a look at this issue as it has been labeled with an integration (vicare) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of vicare can trigger bot actions by commenting:

• @home-assistant close Closes the issue.
• @home-assistant rename Awesome new title Renames the issue.
• @home-assistant reopen Reopen the issue.
• @home-assistant unassign vicare Removes the current integration label and assignees on the issue, add the integration domain after the command.
• @home-assistant add-label needs-more-information Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue.
• @home-assistant remove-label needs-more-information Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

_{^{(message by CodeOwnersMention)}}

---

vicare documentation
vicare source
_{^{(message by IssueLinks)}}

[issue-triage-workflows]

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates.
Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍
This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

[CFenner]

@NoRi2909 could you elaborate the difference between PV and solar again? Is one electric the other thermic?
Do you have both in your system?

[NoRi2909]

Hi @CFenner, yes "PV" is photovoltaics, while "Solar" is thermal solar. Viessmann offers both, so there can be combined systems, using thermal solar for hot water and PV for electricity generation.

The key names clearly differentiate both, starting with either photovoltaic_ or solar_ but the friendly names lost that.
So you just need to change the friendly names on the first four from "Solar …" to "PV …".

This ensures that translators also pick different names to keep them apart and we no longer have two sensor entities named "Solar energy production today" or "Solar energy production this month".

[CFenner]

But in general, these entities cannot be displayed on the same device, one is only for heating systems the other for ess.

Are you also fine with *Solarthermic * and *Photovoltaic *? Then it's even more explicit.

[NoRi2909]

I don't think it's necessary to change "Solar" to "Solarthermic" as this would no longer match the key name either. And this is making them even longer than now.

For the photovoltaic_ you can decide whether you want to keep the full name or rather abbreviate it like with "DHW".

I personally would prefer the shorter "PV" as these sensor names don't fit the default width on the Device info page, or on mobile devices. Anyone with PV one the roof will know what this abbreviation means.