-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Addition of floating module #1937
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
Comments
Overall structure looks OK to me. |
Yes, the I only know of wave spectra for oceans, as the waves in inland-water bodies are generally small. I think that most disturbances in lakes, ponds, canals and the like are generally caused by human actions (e.g. ships) or random actions so they are harder to model. I have finally had the time to work on this. I have created a fork of pvlib in my github. The files to check are I'd like your opinion on the coding structure before merging it into the main code. I'm aware that the files are not commented and there are no tests, but as the structure may be subjected to changes, I didn't want to get into those details yet. There are also some functions that work with random values, but for now are set to ones to improve the "testing". |
As shown at the 2023 European PVPMC, people have interest on modelling the inclination angles of a floating PV system. I have developed and published a Matlab code for that here that I would like to include to pvlib.
In summary, the code consists of the following steps:
Since the code is in matlab, it should be organized differently in python. Below is my first proposal for the organization in its own
floating.py
module.This would ensure that sequential functions are called once while allowing individual functions if needed.
What do you think? Do you have any suggestions for improvement?
The text was updated successfully, but these errors were encountered: