pvlib / pvlib-python

A set of documented functions for simulating the performance of photovoltaic energy systems.
https://pvlib-python.readthedocs.io
BSD 3-Clause "New" or "Revised" License
1.16k stars 980 forks source link

Fronius Symo Advanced 17.5-3-M missing #2145

Closed andreas-bulling closed 1 month ago

andreas-bulling commented 1 month ago

the above inverter is missing in the database

https://www.fronius.com/de/solarenergie/installateure-partner/technische-daten/alle-produkte/wechselrichter/fronius-symo-advanced/symo-advanced-17-5-3-m

echedey-ls commented 1 month ago

Do you mean this one? https://github.com/NREL/SAM/blob/1555ed5bdf259c2f373a61d34bd5033e37c6e2c9/deploy/libraries/CEC%20Inverters.csv#L562

Just so you know, pvlib-python only provides some databases distributed by the project above and some others. If the inverter is not there, you will need to wait until some lab characterizes it.

Two things:

Maintainers, have any of you came across this error? I read long time ago you modify the databases before distributing them through pvlib.

andreas-bulling commented 1 month ago

This is the "old" model - further below are inverters from the latest "Advanced" series but specifically the 17.5 is missing.

I had reported this elsewhere and was redirected here. I will now open an issue with SAM directly. Thanks for the hint.

kandersolar commented 1 month ago

Unfortunately, SAM will not be able to help with this either. pvlib distributes the tables from SAM, but SAM is also merely distributing them from another source -- the California Energy Commission (CEC). See here: https://www.energy.ca.gov/programs-and-topics/programs/solar-equipment-lists

This may also be helpful: https://pvlib-python.readthedocs.io/en/stable/user_guide/faq.html#the-cec-table-doesn-t-include-my-module-or-inverter-what-should-i-do

kandersolar commented 1 month ago

@echedey-ls you have to use the "raw" file URL, like this:

retrieve_sam(path="https://raw.githubusercontent.com/NREL/SAM/patch/deploy/libraries/CEC%20Inverters.csv")
kandersolar commented 1 month ago

@andreas-bulling you could also check older versions of the SAM database. Sometimes older devices get removed from the database, so older versions may have the device you're looking for.

In any case, this is not really a pvlib problem, so I will close this issue. We already have #1345 to track the general issue of pvlib's tables being out of date.

cwhanse commented 1 month ago

@andreas-bulling if you can get the required information (perhaps digitize the CEC efficiency curves from the spec sheet), you can use pvlib.inverter.fit_sandia to get model coefficients. The method in this function is the same as used by the SAM team to derive model coefficients from the CEC data. An inverter enters the CEC data list only when someone submits a test report to the CEC, and the inverter meets certain efficiency standards.

cwhanse commented 1 month ago

@andreas-bulling I think the data you need are at the link you posted. I am not certain of the DC voltages used for the European efficiency values, but those look very similar to the CEC's measurement protocol.

andreas-bulling commented 1 month ago

@cwhanse Thanks for your reply/help. I don't actually want to use the library directly. I want to use EMHASS (https://github.com/davidusb-geek/emhass-pvlib-database) but noticed that I can't because this inverter is missing. The EMHASS author recommended to open an issue here.

cwhanse commented 1 month ago

Maintainers, have any of you came across this error? I read long time ago you modify the databases before distributing them through pvlib.

I haven't. Sometimes the SAM team changes the format of that table between releases, and pvlib has to adapt.

This whole process (CEC tables -> model fitting -> SAM tables -> pvlib distributions) needs to be re-engineered for the benefit of both SAM and pvlib users. The SAM team has been open to cooperating.

0-k commented 1 month ago

Piggy-backing this discussion, I noticed that the cached version of the CEC tables is more than 5 years old data/sam-library-cec-inverters-2019-03-05.csv and data/sam-library-cec-modules-2019-03-05.csv. Is this something that could be updated on the newest SAM release here, which is less than 1 year old? I could assist with that.

cwhanse commented 1 month ago

Piggy-backing this discussion, I noticed that the cached version of the CEC tables is more than 5 years old data/sam-library-cec-inverters-2019-03-05.csv and data/sam-library-cec-modules-2019-03-05.csv. Is this something that could be updated on the newest SAM release here, which is less than 1 year old? I could assist with that.

I suppose we should update those cached files.

I don't see a re-engineering of the CEC-SAM-pvlib parameter process on the horizon, unfortunately. #1345 #769 are relevant.