8000 DC Ohmic Losses for pvsystem and modelchain by ncroft-b4 · Pull Request #1168 · pvlib/pvlib-python · GitHub
[go: up one dir, main page]
Skip to content

DC Ohmic Losses for pvsystem and modelchain #1168

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.

Sign up for GitHub

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

Merged
merged 17 commits into from
Mar 16, 2021
Merged

DC Ohmic Losses for pvsystem and modelchain #1168

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
5c79590
Support functions for ohmic losses
ncroft-b4 Jan 25, 2021
b6fa6f6
PVSystem dc ohmic losses and tests
ncroft-b4 Feb 15, 2021
8b6769c
ModelChain dc ohimc losses as tests
ncroft-b4 Feb 15, 2021
18228de
Merge branch 'master' into redevelopment
ncroft-b4 Feb 15, 2021
ab5ecff
updates to api.rst for dc_ohmic_losses
ncroft-b4 Feb 15, 2021
f63a24b
Docstring formatting updates
ncroft-b4 Feb 24, 2021
0db5afe
make dc_ohms_from_percent work with other dc models
ncroft-b4 Feb 24, 2021
a8a911e
format fixes
ncroft-b4 Feb 24, 2021
3c7b85d
support for multiple arrays
ncroft-b4 Feb 24, 2021
8749450
added tests for new module parameters
ncroft-b4 Feb 24, 2021
4500bc7
modelchain tests for multiple array support
ncroft-b4 Feb 24, 2021
547f6c1
fix api, document render, and use explict variables
ncroft-b4 Mar 5, 2021
68857ab
resolve conflict and small format fix
ncroft-b4 Mar 5, 2021
26be1fe
dc_ohmic_losses as results attribute
ncroft-b4 Mar 5, 2021
f96ac03
fix tests and None model
ncroft-b4 Mar 5, 2021
ba92da6
remove none as model option
ncroft-b4 Mar 15, 2021
19728cd
remove none from tests
ncroft-b4 Mar 15, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
resolve conflict and small format fix
  • Loading branch information
@ncroft-b4
ncroft-b4 committed Mar 5, 2021
commit 68857abf63b97e7580cbe0797cfe8f4c05e27c12
2 changes: 0 additions & 2 deletions pvlib/modelchain.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -429,8 +429,6 @@ def __init__(self, system, location,
self.dc_ohmic_model = dc_ohmic_model
self.losses_model = losses_model

self.orientation_strategy = orientation_strategy

self.weather = None
self.times = None

Expand Down
6 changes: 3 additions & 3 deletions pvlib/pvsystem.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1408,17 +1408,17 @@ def dc_ohms_from_percent(self):

CEC:

* `self.module_parameters["V_mp_ref"]`,
* `self.module_parameters["V_mp_ref"]`
* `self.module_parameters["I_mp_ref"]`

SAPM:

* `self.module_parameters["Vmpo"]`,
* `self.module_parameters["Vmpo"]`
* `self.module_parameters["Impo"]`

PVsyst-like or other:

* `self.module_parameters["Vmpp"]`,
* `self.module_parameters["Vmpp"]`
* `self.module_paramete 4586 rs["Impp"]`

Other array parameters that are used are:
Expand Down
0