Skip to content
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

template/ac-elwa-e: Correct soc and rename file and description #12604

Merged
merged 3 commits into from
Mar 3, 2024

Conversation

stumbaumr
Copy link
Contributor

My-PV has various products (https://www.my-pv.com/de/produkte/ ), I can only test and verify the old AC ELWA-E.

evcc trace output:

[http  ] TRACE 2024/03/02 17:28:34 GET http://192.168.178.23/data.jsn
[http  ] TRACE 2024/03/02 17:28:34 {
"device":"AC ELWA-E",
"fwversion":"00206.02",
"status":3,
"power":0,
"boostpower":0,
"temp1":185,
"ww1target":597,
"boostactive":0,
"legboostnext":"off",
"loctime":"17:28:11",
"unixtime":1709396891,
"ctrlstate":"Conn. to Fronius. P Grid=689",
"blockactive":0,
"meter1_id":null,
"meter1_ip":"null",
"meter2_id":null,
"meter2_ip":"null",
"meter3_id":null,
"meter3_ip":"null",
"meter4_id":null,
"meter4_ip":"null",
"meter5_id":null,
"meter5_ip":"null",
"meter6_id":null,
"meter6_ip":"null",
"meter_ss":null,
"meter_ssid":"null",
"surplus":-689,
"m0sum":-689,
"m0l1":null,
"m0l2":null,
"m0l3":null,
"m0bat":0,
"m1sum":152,
"m1l1":null,
"m1l2":null,
"m1l3":null,
"m1devstate":0,
"m2sum":null,
"m2l1":null,
"m2l2":null,
"m2l3":null,
"m2soc":null,
"m2state":null,
"m2devstate":null,
"m3sum":null,
"m3l1":null,
"m3l2":null,
"m3l3":null,
"m3soc":null,
"m3devstate":3,
"m4sum":null,
"m4l1":null,
"m4l2":null,
"m4l3":null,
"m4devstate":null,
"ecarstate":"null",
"ecarboostctr":null,
"mss2":"null",
"mss3":"null",
"mss4":"null",
"mss5":"null",
"mss6":"null",
"mss7":"null",
"mss8":"null",
"mss9":"null",
"mss10":"null",
"mss11":"null",
"tempchip":30,
"cur_ip":"192.168.178.23",
"cur_sn":"255.255.255.0",
"cur_gw":"192.168.178.1",
"cur_dns":"192.168.178.1",
"cloudstate":4,
"debug_ip":"0.0.0.0"
}

temp1 is the measured temperature in Celcius with a factor of 0.1.

I would really use the name shown from the website and in the status json.

stumbaumr and others added 3 commits March 2, 2024 17:39
My-PV has currently the following products:
- ELWA (DC P2H heater directly from solar power)
- AC ELWA 2 (new version of AC P2H heater using a smartmeter connection)
- AC-THOR (P2H 1 phase control device)
- AC-THOR 9s (P2H 3 phase control device)
- outdated: AC ELWA-E

Source: https://www.my-pv.com/de/produkte/

I only have AC ELWA-E.
@andig andig merged commit 85b3da6 into evcc-io:master Mar 3, 2024
6 checks passed
@andig
Copy link
Member

andig commented Mar 3, 2024

Klasse, danke!

@pinselman
Copy link

Hello, can you add a configuration for AC ELWA 2 (the new heating product of my-PV)?
With the AC-ELWA-E configuration I get a parse error because the format of data.jsn has changed.
The attribute for power is power_elwa2.
There is a second temperature value (temp2). However, installing the sensor is optional.

{
"device":"AC ELWA 2",
"fwversion":"e0000901",
"psversion":"ep105",
"coversion":"ec103",
"fsetup":0,
"p1_s":0,
"p1_v":"null",
"p2_s":240,
"p2_v":"0000901",
"p_co_s":255,
"p_co_v":"null",
"p_ps_s":255,
"p_ps_v":"null",
"power_system":null,
"screen_mode_flag":1,
"power_elwa2":3122,
"power_solar":3122,
"power_grid":0,
"power1_solar":3122,
"power1_grid":0,
"power2_solar":0,
"power2_grid":0,
"power3_solar":0,
"power3_grid":0,
"rel1_out":0,
"rel_selv":0,
"temp1":594,
"temp2":592,
"boostactive":0,
"legboostnext":"null",
"date":"07.03.24",
"loctime":"17:37:27",
"unixtime":1709829447,
"uptime":0,
"ctrlstate":"Modbus multiple Write received",
"blockactive":0,
"meter1_id":null,
"meter1_ip":"null",
"meter2_id":null,
"meter2_ip":"null",
"meter3_id":null,
"meter3_ip":"null",
"meter4_id":null,
"meter4_ip":"null",
"meter5_id":null,
"meter5_ip":"null",
"meter6_id":null,
"meter6_ip":"null",
"meter_ss":null,
"meter_ssid":"null",
"surplus":null,
"m0sum":null,
"m0l1":null,
"m0l2":null,
"m0l3":null,
"m0bat":null,
"m1sum":null,
"m1l1":null,
"m1l2":null,
"m1l3":null,
"m1devstate":null,
"m2sum":null,
"m2l1":null,
"m2l2":null,
"m2l3":null,
"m2soc":null,
"m2state":null,
"m2devstate":null,
"m3sum":null,
"m3l1":null,
"m3l2":null,
"m3l3":null,
"m3soc":null,
"m3devstate":null,
"m4sum":null,
"m4l1":null,
"m4l2":null,
"m4l3":null,
"m4devstate":null,
"ecarstate":"null",
"ecarboostctr":null,
"mss2":"null",
"mss3":"null",
"mss4":"null",
"mss5":"null",
"mss6":"null",
"mss7":"null",
"mss8":"null",
"mss9":"null",
"mss10":"null",
"mss11":"null",
"volt_mains":217,
"volt_aux":4,
"freq":49970,
"temp_ps":455,
"fan_speed":0,
"ps_state":0,
"ctrl_errors":0,
"warnings":0,
"fwversionlatest":"e0000901",
"coversionlatest":"ec103",
"psversionlatest":"ep105",
"upd_state":0,
"ps_upd_state":0,
"co_upd_state":0,
"cur_eth_mode":1,
"wifi_signal":3,
"wifi_list":[{"ssid":"","signal":0},{"ssid":"","signal":0},{"ssid":"","signal":0},{"ssid":"","signal":0},{"ssid":"","signal":0},{"ssid":"","signal":0},{"ssid":"","signal":0},{"ssid":"","signal":0},{"ssid":"","signal":0},{"ssid":"","signal":0},{"ssid":"","signal":0},{"ssid":"","signal":0},{"ssid":"","signal":0},{"ssid":"","signal":0},{"ssid":"","signal":0},{"ssid":"","signal":0}],
"cur_ip":"192.168.178.120",
"cur_sn":"255.255.255.0",
"cur_gw":"192.168.178.1",
"cur_dns":"192.168.178.1",
"cloudstate":4,
"debug_ip":"0.0.0.0"
}

@stumbaumr stumbaumr deleted the patch-1 branch March 7, 2024 23:05
stumbaumr added a commit to stumbaumr/evcc that referenced this pull request Mar 7, 2024
andig pushed a commit that referenced this pull request Mar 8, 2024
thierolm pushed a commit to thierolm/evcc that referenced this pull request Mar 10, 2024
thierolm pushed a commit to thierolm/evcc that referenced this pull request Mar 10, 2024
@pinselman
Copy link

Thanks for the fast implementation.

thierolm pushed a commit to thierolm/evcc that referenced this pull request Mar 19, 2024
thierolm pushed a commit to thierolm/evcc that referenced this pull request Mar 19, 2024
naltatis pushed a commit that referenced this pull request Mar 26, 2024
naltatis pushed a commit that referenced this pull request Mar 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants