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

remove pvsystem.systemdef function #965

Closed
wholmgren opened this issue May 15, 2020 · 1 comment · Fixed by #1008
Closed

remove pvsystem.systemdef function #965

wholmgren opened this issue May 15, 2020 · 1 comment · Fixed by #1008

Comments

@wholmgren
Copy link
Member

Does anyone use this function? Does anyone have use case for it? I'm inclined to remove it without deprecation in 0.8.

Another option is to move it to pvlib.iotools.tmy.

@wholmgren wholmgren added the api label May 15, 2020
@wholmgren wholmgren added this to the 0.8.0 milestone May 15, 2020
@mikofski
Copy link
Member

mikofski commented May 15, 2020

I don't use it, I had to search pvlib for systemdef

code:

relevant issues:

relevant tests:

docs:
api.rst

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

Successfully merging a pull request may close this issue.

2 participants