-
Notifications
You must be signed in to change notification settings - Fork 79
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
PVLib DBs append custom EMHASS modules and inverters #419
Conversation
I could look into automating this procedure also. But that may be a future goal. |
Good idea. |
Hi David The parameters could be obtained either from manufacturers directly if they are willing to share / provide or simply google / chatgpt as most parameters are publicly searchable - all but certs such as C0, C1, C2, C3 which can be approximated for now. Ive just emailed Sungrow to confirm these so we can have a better idea if what Ive found is correct. That will update Sungrow inverters to new data and make emhass much much more accurate. Raf |
@davidusb-geek @RafAustralia , I have added an git action that will now (or should) automatically update cec_inverters.pbz2 and cec_modules.pbz2 files. |
Sorry about all the commits. Thats what I get for going too fast. |
PR is good to go 👍 |
Let me just say that is bloody exciting! |
The parameters are obtained my using real measurements in a laboratory test bench. So yes either the manufacturer or a lab entity as the NREL. These are not easy parameters to obtain. But yes they can be approximated. |
Great! |
It seems that the "Update EMHASS PVLib databases" job is failing when trying to git push at the end |
@davidusb-geek 's completely fine. Since i have already updated them it will fail |
I might look into the future to edit the action so that it passes even tho it failed |
100% agree. |
Ability to add inverters / modules that otherwise wont be added to SAM.
(Will review my work tommorow when im more awake)
Discussed by @RafAustralia,
Related to issue: NREL/SAM#1989