Skip to content

Tutorial: pvsystem.ipynb #267

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
cdeline opened this issue Nov 18, 2016 · 3 comments · Fixed by #281
Closed

Tutorial: pvsystem.ipynb #267

cdeline opened this issue Nov 18, 2016 · 3 comments · Fixed by #281

Comments

@cdeline
Copy link
Contributor

cdeline commented Nov 18, 2016

I'm getting some errors when running pvsystem.ipynb in the PVLib tutorials directory. Specifically code block 17 should be replaced with
pacs = pvsystem.snlinverter(vdcs, pdcs, inverters['ABB__MICRO_0_25_I_OUTD_US_208_208V__CEC_2014_'])

Also, code block 22 is throwing an error at irrad_data = clearsky.ineichen(times_loc, tus.latitude, tus.longitude). It looks like this code example is passing local datetime into clearsky.ineichen, rather than apparent_zenith angle in degrees.

I'm using the latest pvlib 0.4.1 python 2.7.

Chris Deline

@wholmgren
Copy link
Member

Thanks for pointing that out and sorry for the confusion. We made some API changes in 0.4 and neglected to update many of the jupyter/ipython tutorials. The online documentation should be up to date: http://pvlib-python.readthedocs.io

Your second error can be fixed with: irrad_data = tus.get_clearsky(times_loc)

@wholmgren
Copy link
Member

This one should work better:

http://nbviewer.jupyter.org/github/wholmgren/pvlib-python/blob/tutorials041/docs/tutorials/pvsystem.ipynb

I'll add this to a future release.

@cdeline
Copy link
Contributor Author

cdeline commented Dec 7, 2016 via email

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