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

replace pvcell with abstract IV-curve API #39

Open
mikofski opened this Issue Jan 12, 2017 · 3 comments

Comments

Projects
None yet
2 participants
@mikofski
Contributor

mikofski commented Jan 12, 2017

replace pvcell with an IV curve backend API that can be changed to another backend for either 5-parameter model or a "pvsyst" like model.

@bmeyers

This comment has been minimized.

Collaborator

bmeyers commented Feb 7, 2017

I was just testing the onediode method in pvlib.pvsystem, and it doesn't appear to have any reverse bias behavior modeled:

image

It seems like we unfortunately can't drop that in as a 1:1 replacement for .calcCell.

@mikofski

This comment has been minimized.

Contributor

mikofski commented Feb 7, 2017

Unfortunately no, but I wasn't expecting that they would have reverse bias because it's not part of the DeSoto or 5-parameter model. However it shouldn't be a problem to drop the forward bias part into pvcell and leave the reverse bias alone:

Icell = Iforward + I reverse
@bmeyers

This comment has been minimized.

Collaborator

bmeyers commented Feb 7, 2017

I see, that makes sense

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