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.

Sign up for GitHub

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

Jump to bottom

Add Smartfox Pro #11353

Closed
wants to merge 14 commits into from
Closed

Add Smartfox Pro #11353

wants to merge 14 commits into from

Conversation

rasenschachmatt
Copy link

This pull request fixes #11318.

Test output from evcc meter with my SMARTFOX Pro (Firmware Version EM2 00.01.07.03) at night (PV power = 0W):

smartfox-grid
Power: 6315W
Energy: 68864.2kWh
Current L1..L3: 15.7A 7.47A 6.35A
Voltage L1..L3: 225V 227V 225V
Power L1..L3: 3.47e+03W 1.48e+03W 1.37e+03W

smartfox-pv
Power: 0W
Energy: 63478.2kWh

smartfox-aux
Power: 0W

@rasenschachmatt rasenschachmatt mentioned this pull request Dec 30, 2023
Closed
@andig
Copy link
Member

andig commented Dec 30, 2023 •
edited
Loading

Sieht super aus, vielen Dank! Würde es nich in smartfox-pro umbenennen, oder passt das vom Produktnamen her nicht?

@andig andig changed the title evcc YAML template for SMARTFOX Pro (Firmware Version EM2 00.01.07.03) Add SMARTFOX Pro (Firmware Version EM2 00.01.07.03) Dec 30, 2023
@rasenschachmatt
Copy link
Author

Sieht super aus, vielen Dank! Würde es nich in smartfox-pro umbenennen, oder passt das vom Produktnamen her nicht?

Von mir aus können wir es gerne umbenennen.

Mein ursprünglicher Gedanke war, dass das bestehende Template smartfox auf der 'all' URI (mit JSON Response) basierte, während das neue die 'values.xml' URI (mit XML Response) verwendet.

Mit welcher Firmware Version die URI des Smartfox Pro gewechselt hat, kann ich aus den Release Notes leider nicht eruieren.

@rasenschachmatt
Copy link
Author

Template heisst nun smartfox-pro.

@premultiply premultiply self-assigned this Dec 31, 2023
@premultiply premultiply added the devices Specific device support label Dec 31, 2023
@premultiply premultiply changed the title Add SMARTFOX Pro (Firmware Version EM2 00.01.07.03) Add Smartfox Pro Dec 31, 2023
@premultiply premultiply marked this pull request as draft December 31, 2023 07:14
@rasenschachmatt rasenschachmatt marked this pull request as ready for review January 1, 2024 01:53
@andig
Copy link
Member

andig commented Jan 3, 2024

Ich würde jetzt vorschlagen, dass wir das konsequent als 1:1 Kopie des alten Templates für EM2 betrachten.

@rasenschachmatt ich kann meine Änderungen nicht pushen:

! [remote rejected] HEAD -> rasenschachmatt/master (permission denied)

Daher weiter in #11450

@andig andig closed this Jan 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andig andig andig left review comments

@premultiply premultiply premultiply left review comments

Assignees

@premultiply premultiply

Labels
devices Specific device support
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Smartfox Template funktioniert nicht mit Smartfox Pro (aktuelle Firmware EM2 00.01.07.03)
3 participants
@rasenschachmatt @andig @premultiply