Skip to content

Inverter Temperature Derate based on Pmax#412

Merged
mjprilliman merged 10 commits into
developfrom
inv_temp_derate_pmax
Nov 6, 2020
Merged

Inverter Temperature Derate based on Pmax#412
mjprilliman merged 10 commits into
developfrom
inv_temp_derate_pmax

Conversation

@mjprilliman

Copy link
Copy Markdown
Collaborator

-Changed inverter derate table to be based on ratio of max output power to rated power rather than efficiency
-Update Y-axis label to reflect the ratio rather than the percentage
-Update default inverter temperature derate table to reflect more realistic examples (resource provided by Will Hobbs from Southern Company: https://files.sma.de/downloads/Temp-Derating-TI-en-15.pdf)
-Related NatLabRockies/ssc#471, closes NatLabRockies/ssc#442

@mjprilliman mjprilliman added this to the 2021 Release milestone Oct 27, 2020
@mjprilliman

Copy link
Copy Markdown
Collaborator Author

@cpaulgilman There will be some documentation changes based on the improvements in this PR most likely.

@cpaulgilman

Copy link
Copy Markdown
Collaborator

Thanks for the documentation heads-up.

@cpaulgilman cpaulgilman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

UI looks good. Table is a bit tricky to use, but functional enough once you understand how it works. Is there a maximum limit to the number of columns in the table?

@mjprilliman

Copy link
Copy Markdown
Collaborator Author

Right now I think the labels stop after 9 columns (four different temp-slope combos). But the editor allows more columns to be generated without the appropriate columns (just called 10 and 11). Should a max number of columns be enforced and if so how?

@cpaulgilman

Copy link
Copy Markdown
Collaborator

I think it's OK not to enforce a maximum -- we can rely on an explanation in Help. Perhaps we could add a UI label with text like "Table supports up to four temperature - slope pairs in each row." either above the table or below it somewhere next to the Update plot button.

@mjprilliman mjprilliman merged commit 89d1d57 into develop Nov 6, 2020
@mjprilliman mjprilliman deleted the inv_temp_derate_pmax branch December 2, 2020 21:09
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

Successfully merging this pull request may close these issues.

Inverter Temperature Derate of Max Power

2 participants