Skip to content

Added GE inverter derate curve for voltage 1300#804

Merged
mjprilliman merged 6 commits into
developfrom
inv_derate_high_voltage
Dec 1, 2021
Merged

Added GE inverter derate curve for voltage 1300#804
mjprilliman merged 6 commits into
developfrom
inv_derate_high_voltage

Conversation

@mjprilliman

Copy link
Copy Markdown
Collaborator

Description

Added GE inverter derate data for inverter with higher operating voltage (Vdc). Added this curve to existing inverter derate curves.

Source: https://www.ge.com/renewableenergy/sites/default/files/related_documents/lv5plus-solar-inverter-datasheet.pdf

Fixes #803
See #NatLabRockies/ssc#442

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Checklist:

If you have added a new compute module in a SSC pull request related to this one, be sure to check the Process Requirements.

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules
  • I have checked my code and corrected any misspellings

@mjprilliman mjprilliman added this to the SAM Fall 2021 Release milestone Nov 24, 2021
@mjprilliman mjprilliman self-assigned this Nov 24, 2021
@mjprilliman mjprilliman requested a review from dguittet November 24, 2021 17:11

@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.

Looks good to me.

One question: Should versions.lk overwrite the old temperature derate table with the new one? I think it should only do that if the old table has default values, i.e., was not modified by user -- not sure if it's worth the effort to code that comparison.

@dguittet

Copy link
Copy Markdown
Collaborator

I think this combination of two different inverter derate curve specs (1 high voltage line from 1 high voltage inverter and the rest from a mid-voltage inverter) looks kind of odd. My intended but missed goal for the original example was to show how to input multiple lines for different voltages, not really to provide a single "default" set that would apply to most situations since I don't know what that would even be. I'd propose something like the following for the default since it shows the capability, kind of extrapolates the behavior of the high-voltage inverter, and very often has little impact so the users won't get anything they weren't anticipating.

image

@mjprilliman

Copy link
Copy Markdown
Collaborator Author

I think this combination of two different inverter derate curve specs (1 high voltage line from 1 high voltage inverter and the rest from a mid-voltage inverter) looks kind of odd. My intended but missed goal for the original example was to show how to input multiple lines for different voltages, not really to provide a single "default" set that would apply to most situations since I don't know what that would even be. I'd propose something like the following for the default since it shows the capability, kind of extrapolates the behavior of the high-voltage inverter, and very often has little impact so the users won't get anything they weren't anticipating.

image

I would not be opposed to these default curves. What reference did you get these curves from?

@dguittet

Copy link
Copy Markdown
Collaborator

Well... I just made up the 390 V curve based on the 1600 V one. 😜

@mjprilliman

Copy link
Copy Markdown
Collaborator Author

Well... I just made up the 390 V curve based on the 1600 V one. 😜

Aah I understand now I should've read your first comment more closely. I'm not sure what the best path forward would be. @cpaulgilman should we consider having different defaults for the different default system sizes?

@cpaulgilman

Copy link
Copy Markdown
Collaborator

Well... I just made up the 390 V curve based on the 1600 V one. 😜

Aah I understand now I should've read your first comment more closely. I'm not sure what the best path forward would be. @cpaulgilman should we consider having different defaults for the different default system sizes?

I don't think there is a perfect quick solution for a last minute fix before the release. The user-reported issue on the forum was caused when the user chose an inverter with VDCmax greater than the 800 VDC maximum value in the temperature derate table -- that would not be solved by having different default values.

One solution would be to display a message when that happens like "Maximum inverter DC voltage is greater than the highest voltage in the temperature derate table. See the inverter manufacturer datasheet for information about temperature derating and change the derate table accordingly."

Some thoughts:

  1. Ideally, the inverter library would include inverter temperature derate data.
  2. We could write a callback to set the default temperature derate curve when the library choice changes (or when the maximum DC voltage changes so it works with the datasheet option), perhaps for three or four ranges of inverter maximum rated DC voltage.
  3. Darice's made up curves probably work well, but are hard to justify. The current default curves, which I think are from https://files.sma.de/downloads/Temp-Derating-TI-en-15.pdf for smaller inverters has a smaller slope at high temps than the larger inverters, and the made up curves do not reflect that.

@dguittet

Copy link
Copy Markdown
Collaborator

I'm not sure how commonly reported this inverter derate data is. If we were to set up something as a default for all inverters, I would go the route of one of minimal complexity and with minimal effect on the simulation. Since it's there as an illustration not as a meaningful input. Just my 2 cents though.

@mjprilliman

Copy link
Copy Markdown
Collaborator Author

Changing the default to one inverter curve to cover wide voltage range with details on how to enter multiple curves available in help

Reference: https://www.ge.com/renewableenergy/sites/default/files/related_documents/lv5plus-solar-inverter-datasheet.pdf

@mjprilliman mjprilliman merged commit 654af42 into develop Dec 1, 2021
@mjprilliman mjprilliman deleted the inv_derate_high_voltage branch December 1, 2021 00:23
@cpaulgilman cpaulgilman added the added to release notes PR and/or issue has been added to release notes for a public release label Dec 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

added to release notes PR and/or issue has been added to release notes for a public release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update default inverter temperature derate curves for large inverters

3 participants