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

Feature/update if w hawc #437

Merged
merged 4 commits into from
Apr 17, 2020
Merged

Conversation

ewquon
Copy link
Contributor

@ewquon ewquon commented Apr 16, 2020

Complete this sentence
THIS PULL REQUEST IS READY TO MERGE

Feature or improvement description
Add an additional option to the WindProfile parameter: -1 is allowable so that the mean profile is not added on. Use case would be when a user wants to directly supply inflow planes that include both the mean wind and turbulence, e.g., when using sampled output from SOWFA and using it as inflow data.

@rafmudaf rafmudaf changed the base branch from master to dev April 16, 2020 14:55
@andrew-platt
Copy link
Collaborator

andrew-platt commented Apr 17, 2020

Also added the ability to shift the HAWC wind. This is added into the InflowWind input files as the last line in the Mean wind profile parameters (added to HAWC-format files)

         0    InitPosition(x) - Initial offset in +x direction (shift of wind box)

This an optional input (to maintain backwards compatibility). We updated the InflowWind input files to include this line just so we don't forget about it at a later date.

Copy link
Contributor Author

@ewquon ewquon left a comment

Choose a reason for hiding this comment

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

These changes look good to me.

@rafmudaf rafmudaf merged commit 1365da7 into OpenFAST:dev Apr 17, 2020
andrew-platt added a commit to andrew-platt/openfast that referenced this pull request Jun 23, 2020
@rafmudaf rafmudaf mentioned this pull request Sep 11, 2020
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.

3 participants