Skip to content

Parameter descriptions for retrieve_sam #838

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

Open
toddkarin opened this issue Dec 13, 2019 · 3 comments
Open

Parameter descriptions for retrieve_sam #838

toddkarin opened this issue Dec 13, 2019 · 3 comments

Comments

@toddkarin
Copy link

The pvlib.pvsystem.retrieve_sam function pulls information on modules, but the different keys don't have descriptions. For example, I don't know what 'A_c' means in the CECMod database. Can these various fields be described?

@toddkarin
Copy link
Author

toddkarin commented Dec 13, 2019

By sleuthing into the CEC database 'A_c' seems to be the module area in m^2, but it would still be nice to have this documented in PVLIB.

For example Beta_oc is in %/C in the CEC module database, but in V/C when returned from pvlib.pvsystem.retrieve_sam.

@toddkarin
Copy link
Author

@cwhanse Would this be a good opportunity for incorporating pv-terms in order to describe the various parameters? It may also require a relabeling of the CEC values.

@cwhanse
Copy link
Member

cwhanse commented Jul 27, 2020

Yes, it is timely to consider changing terms and improving the documentation. There's a discussion of refactoring the current retrieve_sam into the pvlib.iotools module. With new functions in iotools we would deprecate the current retrieve_sam for a while, to make transition easier.

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

No branches or pull requests

2 participants