Skip to content

Using cec-modules with sapm #122

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
uvchik opened this issue Feb 17, 2016 · 4 comments
Closed

Using cec-modules with sapm #122

uvchik opened this issue Feb 17, 2016 · 4 comments
Milestone

Comments

@uvchik
Copy link
Contributor

uvchik commented Feb 17, 2016

As I understand it, it is not possible to use the pvlib.pvsystem.sapm function with the cec-modules-library.

Is this right?

If it is right, is it possible to write a hint somewhere in the documentation.

http://pvlib-python.readthedocs.org/en/latest/pvlib.html#pvlib.pvsystem.sapm

@cwhanse
Copy link
Member

cwhanse commented Feb 17, 2016

The CEC library doesn’t contain the parameters that are needed for the SAPM. It only has parameters for the CEC performance model.

NREL might (I’m not sure) have added functionality to translate from CEC model parameters to SAPM parameters. I know PVsyst has done this, but the translation is suspect in my opinion. It works by computing modeled IV curves (using the PVsyst model), picking off Isc, Vmp/Imp and Voc, then fitting the SAPM equations to these modeled points. So you get a SAPM model which emulates the PVsyst model including any biases, rather than measured performance.

Cliff

@wholmgren
Copy link
Member

Sure, let's add a hint to the documentation. Feel free to make a PR, otherwise I'll get to it sometime in the next few weeks.

@wholmgren wholmgren added this to the 0.3 milestone Feb 17, 2016
@uvchik
Copy link
Contributor Author

uvchik commented Feb 18, 2016

See PR #123.

@wholmgren
Copy link
Member

Closed by #123

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants