Skip to content

add temperature.sapm_cell_from_module#938

Merged
CameronTStark merged 9 commits into
pvlib:masterfrom
cwhanse:cell_from_mod
Apr 7, 2020
Merged

add temperature.sapm_cell_from_module#938
CameronTStark merged 9 commits into
pvlib:masterfrom
cwhanse:cell_from_mod

Conversation

@cwhanse

@cwhanse cwhanse commented Mar 19, 2020

Copy link
Copy Markdown
Member

@cwhanse cwhanse added this to the 0.7.2 milestone Mar 19, 2020
Comment thread pvlib/temperature.py
Comment thread pvlib/temperature.py Outdated

@mikofski mikofski left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks Cliff!

@wholmgren wholmgren left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM pending minor fixes.

I think @adriesse had some similar thoughts about this topic in an earlier issue or PR.

Comment thread pvlib/temperature.py Outdated
Comment thread pvlib/temperature.py Outdated
Comment thread pvlib/temperature.py
cwhanse and others added 2 commits March 23, 2020 09:19
Co-Authored-By: Will Holmgren <william.holmgren@gmail.com>
@CameronTStark

Copy link
Copy Markdown
Contributor

Just FYI, I re-ran the two failing conda_windows tests and they came back passing.

I did so while comparing CI failures in PRs to compare to the ones that popped up for #937 (spoiler: they were different).

Comment thread docs/sphinx/source/whatsnew/v0.7.2.rst Outdated
@CameronTStark

Copy link
Copy Markdown
Contributor

Sorry, I merged #937 which caused another conflict here. If you don't mind re-un-conflicting @cwhanse.

@cwhanse cwhanse changed the title add temperature.sapm_cell_to_module add temperature.sapm_cell_from_module Apr 7, 2020
@CameronTStark CameronTStark merged commit 06f8f6e into pvlib:master Apr 7, 2020
@CameronTStark

Copy link
Copy Markdown
Contributor

Thank you @cwhanse!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ENH] cell temperature from backside of module temperature measurements

4 participants