-
Notifications
You must be signed in to change notification settings - Fork 51
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
Bug: Incorrect list of supported modules / inverters in docs #145
Comments
Hi, That's just not the case. That's what it is for PVLib. I hope that there will be another method with a more updated database. But there isn't and is up to PVLib developers to update this. Maybe you can open an issue there to ask for this update. |
Oh, you are correct. That was very suprising and I think the pvlib documentation is insufficient in this case. However, looking at the pvlib source you linked, it should be possible to supply your own csv file, right? ie by only passing a path to a cvc file, rather that passing a name.
I am not stuck in any way I was just confused on which file was actually used. Thanks for getting back to me! |
Yes if we can put together un updated version of the CSV file then maybe we can pass it directly yes. it can be an option. Now where can we find this updated version. We need to keep the same formatting as the current CSV file. |
As I mentioned in the issue, pvlib refers to files here where it seems the latest update of the list is for fall 2023. It seems to me that the format of the inverter file has somewhat changed, but the one for the modules seems to be at least similar. Whether any of these potential format changes are supported by pvlib or not i do not know.. |
This approach would however require someone to manually update the csv file on new lists. It could of course be retreived from the source by supplying a URL instead, but that is probably a bad idea since it would rely on an internet connection and on the fact that pvlib supports all future formats of the file. |
I didn't saw that option to pass a CSV path and from what I can see that updated CSV has the same format except for the last column. I think that we could use that. I will try it and if it works I will include it in the next release. |
Cool thanks! |
Closing issue as invalid, see this comment |
I reopened this issue because we could possible update the CSV files stored in EMHASS to those more updated files from SAM. But this need to be tested. |
I made the needed changes for this. It will be available in the following releases. |
Released on latest version. |
Hi,
According to the the documentation the list of modules / inverters supported are found here and here.
However, as far as I can tell inverter/modules are retrieved from the oline SAM database. See snippet below
emhass/src/emhass/forecast.py
Line 413 in 5691360
Which indicates, according to pvlib docs that the online database is used.
The actual list of supported modules are located here
Thanks in advance
The text was updated successfully, but these errors were encountered: