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

Update Fronius GEN24 template #4510

Merged
merged 1 commit into from
Sep 22, 2022
Merged

Update Fronius GEN24 template #4510

merged 1 commit into from
Sep 22, 2022

Conversation

premultiply
Copy link
Member

@premultiply premultiply commented Sep 18, 2022

  • add Primo Gen24 Plus to description
  • add total energies from grid and inverter
  • autodetect grid meter model
  • remove integer setting from configure

Fixes #4434 (comment)

@premultiply premultiply added enhancement New feature or request devices Specific device support labels Sep 18, 2022
@premultiply premultiply self-assigned this Sep 18, 2022
@premultiply
Copy link
Member Author

@Macrisu Can you test this?

@Macrisu
Copy link
Contributor

Macrisu commented Sep 18, 2022

How?
Using the nightly build in docker and the fronius gen24 hybrid template?

@thierolm
Copy link
Sponsor Contributor

thierolm commented Sep 18, 2022

Hi @premultiply , ich habe einen Symo GEN24 8.0 Plus,
grid funktioniert,
pv und battery leider nicht: :-(

Es kommt folgender Fehler:

[main  ] INFO 2022/09/18 18:10:28 evcc 0.102 (da9319fb)
[main  ] INFO 2022/09/18 18:10:28 using config file ./evcc_fritzdecttest.yaml
[main  ] FATAL 2022/09/18 18:10:28 cannot create meter 'pv': cannot create meter 'template': cannot create meter 'custom': power: can only have either add/mul or sign

@premultiply
Copy link
Member Author

Huch, was ist denn da passiert?
Das hat aber wohl nichts mit diesem PR an sich zu tun. Oder @andig ?

@thierolm Könntest du so nett sein und grid mit diesem PR nochmal sowohl mit Float als auch Int+SF-Einstellung am WR testen?
Ich möchte Wissen ob die SunSpec-Model-Autoerkennung dabei zuverlässig in beiden Varianten funktioniert.

@andig
Copy link
Member

andig commented Sep 18, 2022

Ups, schon wieder ein Bugfixrelease…

@thierolm
Copy link
Sponsor Contributor

thierolm commented Sep 18, 2022

@premultiply Ich kann leider die Modbus Parameter am WR nicht ändern, weil ich das Techniker Passwort nicht habe. :-(

Bei meiner aktuellen Einstellung in der 0.101.3 steht im template integer auf false. D.h. es müsste im WR auf Float stehen.

@andig
Copy link
Member

andig commented Sep 18, 2022

0.102.1 läuft durch, damit gehts...

@thierolm
Copy link
Sponsor Contributor

Jo, läuft: :-)

thierolm@accffmmt:~/evcc$ ./evcc -c ./evcc_fronius_symo_gen24_test.yaml -l debug meter
[main  ] INFO 2022/09/18 20:29:46 evcc 0.102.1 (91e1ceac)
[main  ] INFO 2022/09/18 20:29:46 using config file ./evcc_fronius_symo_gen24_test.yaml
grid
----
Power:          765W
Current L1..L3: 0.395A 3.16A 0.383A

pv
--
Power: 0W

battery
-------
Power: 0W
SoC:   7%

@thierolm
Copy link
Sponsor Contributor

Nur fehlen da nicht die neuen Energy Werte?

@thierolm
Copy link
Sponsor Contributor

... und der Wegfall des integer Parameters in der Fronius Template Config muss als "Breaking Change" in den Release Notes markiert werden.

@premultiply
Copy link
Member Author

Erst muss ich wissen ob es überhaupt so funktioniert ;)

@andig Stören überflüssige Template-Parameter? Ich dachte bis jetzt die werden ggf. einfach ignoriert.

@premultiply
Copy link
Member Author

Nur fehlen da nicht die neuen Energy Werte?

Du müsstest zum testen fix/fronius-templates Branch auschecken.

@andig
Copy link
Member

andig commented Sep 18, 2022

Ich glaub die stören. Not sure…

@thierolm
Copy link
Sponsor Contributor

Hab' die fix branch jetzt ausgecheckt und einen make laufen lassen.
Da bekomme ich den "breaking" Fehler:

thierolm@accffmmt:~/evcc$ ./evcc -c ./evcc_fronius_symo_gen24_test.yaml -l debug meter
[main  ] INFO 2022/09/18 22:06:43 evcc 0.102.1 (13ac7879)
[main  ] INFO 2022/09/18 22:06:43 using config file ./evcc_fronius_symo_gen24_test.yaml
[main  ] FATAL 2022/09/18 22:06:43 cannot create meter 'grid': cannot create meter 'template': invalid key: integer

@thierolm
Copy link
Sponsor Contributor

Nehme ich den integer: false raus, sehe ich jetzt auch Energy und alle anderen Werte korrekt:

thierolm@accffmmt:~/evcc$ ./evcc -c ./evcc_fronius_symo_gen24_test.yaml -l debug meter
[main  ] INFO 2022/09/18 22:08:42 evcc 0.102.1 (13ac7879)
[main  ] INFO 2022/09/18 22:08:42 using config file ./evcc_fronius_symo_gen24_test.yaml
pv
--
Power:  0W
Energy: 2719.9kWh

battery
-------
Power: 0W
SoC:   6%

grid
----
Power:          587W
Energy:         754.2kWh
Current L1..L3: 0.37A 2.51A 0.544A

@premultiply
Copy link
Member Author

Vielen Dank!

Dann bräuchten wir nur noch einen Gegencheck mit der Einstellung Int+SF.
Wer kann da helfen?

@andig
Copy link
Member

andig commented Sep 18, 2022

Pv Energy verwenden wir nirgendwo

@StefanSchoof

This comment was marked as off-topic.

@Macrisu
Copy link
Contributor

Macrisu commented Sep 19, 2022

Ich habe nur einen Symo 7.0-3-M (nicht gen24), der steht auf int + SF und ich habe testweise mal das template eingebunden:

meters:
  - type: template
    template: fronius-gen24
    usage: pv
    host: pv.fritz.box
    port: 502
    name: pv
go run main.go -l debug meter
[main  ] INFO 2022/09/19 09:57:45 evcc 0.0.1-alpha
[main  ] INFO 2022/09/19 09:57:45 using config file /home/stefan/evcc/evcc.yaml
grid
----
Power: unexpected status: 400

pv
--
Power:  3902W
Energy: 41536.6kWh

Gen24 template does not work with non-gen24 fronius inverter.

@andig
Copy link
Member

andig commented Sep 19, 2022

Soweit ich sehe heissen die aktuellen alle GEN24?

Screenshot 2022-09-19 at 16 29 36

Insofern müsste es im Template Primo/Symo GEN24 sein und der Hinweis nicht-GEN24 Geräte sind nicht unterstützt.

@premultiply
Copy link
Member Author

Das Template heisst hiermit bereits

Fronius

  • Primo GEN24 Plus
  • Symo GEN24 Plus

Noch eindeutiger kann man es doch eigentlich nicht machen, oder?

@StefanSchoof
Copy link
Contributor

Gen24 template does not work with non-gen24 fronius inverter.

I know and dot use this in my normal setup, but nobody with a GEN 24 and Int+SF had done any test. So I thought this is better info then nobody testing it at all.

@andig
Copy link
Member

andig commented Sep 21, 2022

Wissen wir jetzt, was mit Int+SF los ist? Können wir das wirklich entfernen?

Copy link
Member

@andig andig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to verify int+sf!

@Macrisu
Copy link
Contributor

Macrisu commented Sep 21, 2022

Gen24 Inverter int+sf
Screenshot 2022-09-21 at 21 03 23

./evcc -l debug meter
[main  ] INFO 2022/09/21 21:07:55 evcc 0.102.1
[main  ] INFO 2022/09/21 21:07:55 using config file /volume1/docker/evcc31/evcc.yaml
pv_v31
------
Power: 0W

grid_v31
--------
Power:          sunspec: 213:0:W not found
Current L1..L3: sunspec: 213:0:AphA not found

@andig
Copy link
Member

andig commented Sep 21, 2022

@Macrisu you did test the current template from master, not this PR, right? Because 213:AphA is only in master but no longer in this PR. Could you please test the PR, too?

@Macrisu
Copy link
Contributor

Macrisu commented Sep 21, 2022

@Macrisu you did test the current template from master, not this PR, right? Because 213:AphA is only in master but no longer in this PR. Could you please test the PR, too?

I do not know how to test current template...

I've tried also nightly, but error
Screenshot 2022-09-21 at 21 38 53

@andig
Copy link
Member

andig commented Sep 21, 2022

I do not know how to test current template...

I meant current == nightly. We need to find out if this PR works before we can merge it. If you want you can email remote access to info@evcc.io then I could test if this PR works.

@Macrisu
Copy link
Contributor

Macrisu commented Sep 21, 2022

I do not know how to test current template...

I meant current == nightly. We need to find out if this PR works before we can merge it. If you want you can email remote access to info@evcc.io then I could test if this PR works.

Tomorrow I'll setup a rPI for you in my network.

Using the current docker nightly

docker exec -it evcc_nightly evcc -l debug meter
[main  ] INFO 2022/09/21 22:00:04 evcc 0.102.1 (8da58bca)
[main  ] INFO 2022/09/21 22:00:04 using config file /etc/evcc.yaml
pv_v31
------
Power: 0W

grid_v31
--------
Power:          sunspec: 213:0:W not found
Current L1..L3: sunspec: 213:0:AphA not found
meters:
  - name: pv_v31
    type: template
    template: fronius-gen24
    usage: pv
    host: 192.168.1.xx # IP-Adresse oder Hostname
    port: 502 # Port # Optional 
    
   
  - name: grid_v31
    type: template
    template: fronius-gen24
    usage: grid
    host: 192.168.1.xx # IP-Adresse oder Hostname
    port: 502 # Port # Optional 

@andig

This comment was marked as resolved.

@Macrisu

This comment was marked as resolved.

@andig
Copy link
Member

andig commented Sep 22, 2022

Thx. Here's the result for Macrisu:

battery
-------
Power: add[0]: model 160 block 3 point DCW: no such block
.SoC:  sunspec: ChargeState not found

grid
----
Power:          -522W
Energy:         1546.7kWh
Current L1..L3: 1.78A 3.07A 3.27A

pv
--
Power:  2796W
Energy: 3775.0kWh

@Macrisu

This comment was marked as resolved.

@andig
Copy link
Member

andig commented Sep 22, 2022

@Macrisu and your inverter is still on int+sf? @premultiply if yes we're done. The battery errors are expected as no battery attached.

@Macrisu
Copy link
Contributor

Macrisu commented Sep 22, 2022

@Macrisu and your inverter is still on int+sf? @premultiply if yes we're done. The battery errors are expected as no battery attached.

YES - int+sf
Screenshot 2022-09-22 at 11 35 58

@andig andig merged commit 557c4a3 into master Sep 22, 2022
@andig andig deleted the fix/fronius-templates branch September 22, 2022 08:38
@Macrisu
Copy link
Contributor

Macrisu commented Sep 22, 2022

Now can we have a template also for NON-gen 24 Fronius inverters?
Gen24 is Hibrid, non-Gen24 are ongrid by default

Those are the most common now.

The name is:

Fronius Primo - 1 phase
Fronius Symo - 3 phase

I have two of them if you need access.

My config for them (all Symo) is like this:

meters:
- name: grid_v28
  type: modbus
  model: sunspec
  uri: 192.168.1.xx:502
  id: 240
  power: 213:W # sunspec model 213 meter

- name: pv_v28
  type: modbus
  model: sunspec
  uri: 192.168.1.xx:502
  id: 1

@premultiply
Copy link
Member Author

Well, this is simple default SunSpec like many other inverters.
You should be able to remove the power: line from the gridmeter.

@andig
Copy link
Member

andig commented Sep 22, 2022

@premultiply vllt sollten wir Primo/Symo bei Sunspec noch als Brand hinzufügen, wäre Anfängerfreundliche?

@Macrisu
Copy link
Contributor

Macrisu commented Sep 22, 2022

You should be able to remove the power: line from the gridmeter.

meters:
- name: grid_v28
  type: modbus
  model: sunspec
  uri: 192.168.1.xx:502
  id: 240
#  power: 213:W # sunspec model 213 meter

- name: pv_v28
  type: modbus
  model: sunspec
  uri: 192.168.1.xx:502
  id: 1

I've commented # power and is still working! Thank you

@Macrisu
Copy link
Contributor

Macrisu commented Sep 22, 2022

Actually, I think this is better:

meters:
- name: grid_v28
  type: template
  template: sunspec-inverter
  id: 240
  host: 192.168.1.xx
  port: 502
  usage: grid
  modbus: tcpip
  
- name: pv_v28
  type: template
  template: sunspec-inverter
  id: 1
  host: 192.168.1.xx
  port: 502
  usage: pv
  modbus: tcpip 

@premultiply
Copy link
Member Author

@andig Ich überlege mir mal was.
Dann müssten wir aber eigentlich konsequenterweise ziemliche viele WR hinzufügen…

@Macrisu
Copy link
Contributor

Macrisu commented Sep 22, 2022

In my part of the world, the most popular (from expensive to cheap) is:

  • Fronius Gen24
  • Fronius (on-grid)
  • Huawei Hybrid
  • Growatt Hybrid
  • Easun

The new kid on the block promises to be Solis Hybrid, that can work with any kind of battery

mabunixda added a commit to mabunixda/evcc that referenced this pull request Sep 23, 2022
* master:
  Fix resetOnDisconnect overriding default vehicle properties (evcc-io#4551)
  chore: add lint timeout
  more concise button
  Gracefully handle startup errors (evcc-io#4383)
  Add Solarwatt EnergyManager/Pro (evcc-io#4538)
  Fronius Gen24: simplify (evcc-io#4510)
  Telemetry: add green power
  Ford: handle login errors
  Telemetry: log Wh instead of kWh
  Telemetry: send only if above standby power
  Telemetry: switch to bearer auth
  Telemetry: improve naming
  Revert "SE: fix hybrid template (evcc-io#4503)" (evcc-io#4528)
  Allow Credentials (Basic Auth) for manifest.json (evcc-io#4525)
  Experimental: add telemetry for virtual power plant (evcc-io#4343)
  Easee: simplify smart charging
  OpenEVSE: fix null pointer (evcc-io#4515)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devices Specific device support enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants