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

Incorrect units pvlib.ivtools.sdm.fit_desoto #2319

Open
RDaxini opened this issue Dec 3, 2024 · 2 comments · May be fixed by #2328
Open

Incorrect units pvlib.ivtools.sdm.fit_desoto #2319

RDaxini opened this issue Dec 3, 2024 · 2 comments · May be fixed by #2328

Comments

@RDaxini
Copy link
Contributor

RDaxini commented Dec 3, 2024

Is your feature request related to a problem? Please describe.
Units of dEg/dT in pvlib.ivtools.sdm.fit_desoto should be 1/K, not eV/K

Describe the solution you'd like
Correct the units

Additional context
Comment

Marking as good first issue for a new contributor, but I could fix this if no-one does it before 11.2 release if we want it done by then.

@RahulTripathi-04

This comment was marked as off-topic.

@cwhanse
Copy link
Member

cwhanse commented Dec 6, 2024

To clarify: the unit in the docstring is given as eV/K, it should be 1/K, per discussion in #2154. No other change is needed.

@RDaxini RDaxini linked a pull request Dec 9, 2024 that will close this issue
8 tasks
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.

3 participants