netCDF4 missing in dependencies #231

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

Jump to bottom

Closed

emulbreh opened this issue Aug 4, 2016 · 1 comment

Labels

documentation wontfix

Contributor

emulbreh commented Aug 4, 2016 •

edited

Loading

netCDF4 is imported unconditionally in the forecast module but isn't included in setup.py.
https://github.com/pvlib/pvlib-python/blob/master/pvlib/forecast.py#L6

This breaks the documentation build: http://pvlib-python.readthedocs.io/en/latest/forecasts.html

Member

wholmgren commented Aug 17, 2016

See this comment regarding the documentation on readthedocs: #180 (comment) I'm sure it will work on rtd eventually.

I don't want to add netCDF4 to the required dependencies because only a fraction of pvlib users will need it and it's a non-trivial installation.

wholmgren added documentation wontfix labels

wholmgren closed this as completed

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