Skip to content

Wind resource distribution#581

Merged
mjprilliman merged 3 commits into
patchfrom
wind-resource-distribution
Apr 27, 2021
Merged

Wind resource distribution#581
mjprilliman merged 3 commits into
patchfrom
wind-resource-distribution

Conversation

@mjprilliman

@mjprilliman mjprilliman commented Apr 6, 2021

Copy link
Copy Markdown
Collaborator

-Turns off wind resource uncertainty by default
-Hides Uncertainties results tab when uncertainty is not specified
-More meaningful defaults
-Added system capacity to wind power metrics table
-Closes #533
-Closes #547

cpaulgilman and others added 3 commits March 16, 2021 10:48
Addresses #533 and #547:

* Change wind distribution table option label

* More meaningful default value for wind_resource_distribution matrix

* Add 'en_wind_uncertainty' exclusive_var and new Wind Uncertainty None UI form so Uncertainties page can default to descriptive text instead of overwhelming inputs

* Use en_wind_uncertainty to only display p90 Energy metric when uncertainties are enabled

* Add nameplate capacity to Metrics table

* Next commit of Help revisions will include revised description of uncertainty model option and new description of p90 Energy metric
* Use exto wind Uncertainties

@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. I noticed that the Year 1 heat map on the Summary tab is empty, but perhaps that -- will resolve itself when this gets merged into Patch.

@mjprilliman

Copy link
Copy Markdown
Collaborator Author

Looks good. I noticed that the Year 1 heat map on the Summary tab is empty, but perhaps that -- will resolve itself when this gets merged into Patch.

What were the steps you used to get that behavior? The autographs script is supposed to only produce a heatmap for the wind model in the case where the wind resource file is specified, but it should never be empty if patch on ssc is up to date.

@cpaulgilman

Copy link
Copy Markdown
Collaborator

@mjprilliman, I tested with all of the possible wind resource options and the Summary tab heat map is working properly. I did not see an empty graph in these most recent tests.

@cpaulgilman

Copy link
Copy Markdown
Collaborator

@mjprilliman, I think this can be closed.

@mjprilliman mjprilliman merged commit 08a5653 into patch Apr 27, 2021
@cpaulgilman cpaulgilman deleted the wind-resource-distribution branch October 20, 2022 22:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve wind resource option labels, documentation and defaults Wind uncertainty language should indicate you can likely ignore the inputs.

2 participants