Skip to content

Fronius Symo Advanced 17.5-3-M missing #2145

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

Closed
andreas-bulling opened this issue Jul 26, 2024 · 11 comments
Closed

Fronius Symo Advanced 17.5-3-M missing #2145

andreas-bulling opened this issue Jul 26, 2024 · 11 comments

Comments

@andreas-bulling
Copy link

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
Copy link
Contributor

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:

  • pvlib outdated databases do not contain any "Symo"

    Code

    from pvlib.pvsystem import retrieve_sam
    # %%
    d = retrieve_sam(name="CECMod")
    d.columns[d.columns.str.contains("Symo")]
    # %%
    d = retrieve_sam(name="SandiaMod")
    d.columns[d.columns.str.contains("Symo")]

  • Reading from the latest CEC Inverter database, first link on top of comment, gives an error

    from pvlib.pvsystem import retrieve_sam
    retrieve_sam(path="https://github.com/NREL/SAM/blob/patch/deploy/libraries/CEC%20Inverters.csv")
    ParserError: Error tokenizing data. C error: Expected 1 fields in line 40, saw 18

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
Copy link
Author

andreas-bulling commented Jul 26, 2024

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
Copy link
Member

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
Copy link
Member

@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
Copy link
Member

@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
Copy link
Member

cwhanse commented Jul 26, 2024

@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
Copy link
Member

cwhanse commented Jul 26, 2024

@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
Copy link
Author

@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
Copy link
Member

cwhanse commented Jul 26, 2024

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
Copy link

0-k commented Jul 29, 2024

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
Copy link
Member

cwhanse commented Jul 29, 2024

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.

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

No branches or pull requests

5 participants