Skip to content
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

Jump to bottom

update ModelChain documentation regarding CEC/singlediode and adrinverter #480

Closed
wholmgren opened this issue Jun 15, 2018 · 1 comment · Fixed by #489
Closed

update ModelChain documentation regarding CEC/singlediode and adrinverter #480

wholmgren opened this issue Jun 15, 2018 · 1 comment · Fixed by #489
Labels
documentation easy good first issue help wanted
Milestone
0.6.0

Comments

@wholmgren
Copy link
Member

wholmgren commented Jun 15, 2018 •
edited

In #475 @thunderfish24 asked

Is the statement about ModelChain here still valid: "CEC module specifications and the single diode model are not yet supported."?

The statement is not valid and should be fixed. Thanks @thunderfish24 for pointing this out.
@wholmgren wholmgren added this to the 0.6.0 milestone Jun 15, 2018
@wholmgren wholmgren changed the title update ModelChain documentation regarding CEC/singlediode update ModelChain documentation regarding CEC/singlediode and adrinverter Jun 21, 2018
@wholmgren
Copy link
Member Author

The ac_model kwarg states: "'adrinverter' (not implemented)". Need to drop the (not implemented) note.

@wholmgren wholmgren mentioned this issue Jun 24, 2018
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
documentation easy good first issue help wanted
Projects
None yet
Milestone
0.6.0
Development

Successfully merging a pull request may close this issue.

update incorrect statements in ModelChain documentation wholmgren/pvlib-python
1 participant
@wholmgren