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

Various bug fixes and features #27

Merged
merged 80 commits into from
May 12, 2017

Conversation

HaymanConsulting
Copy link
Contributor

This is the rest of the bleeding-edge codebase for OpenFAST. Note: No cmake testing has been performed on this branch prior to issuing the pull request.

Greg Hayman added 30 commits January 12, 2017 12:27
…fix for the tower motions. Also testing an OpenFAST error log to help with automated testing
Conflicts:
	modules-local/fast-library/src/FAST_Mods.f90  - using updated unique version number 8.17.01b-dev
- fixed typo in other state description
- 4Dext: created fatal error is outside the grid bounds, Interp4D doesn't interpolate in time (based on conversation with jmj, we won't interpolate in time because we won't be able to interpolate the last time grid in FAST.Farm)
…s printing of turbine number for FAST.Farm
… output handling draft code, first commit of awae, wakedynamics, and fast-farm into OpenFAST
…D finite differencing bugs, fixed initialization
…AST.Farm to pass data related to VIZ outputs to AWAE
… DLL control option, a seg fault could occur if a fatal error was encountered before the avrSWAP array was allocated.
…line breaks/carriage returns between vector triplets.
…Farm output files to have columns wider than the FAST limit. (10 for FAST, now 14 for FAST.Farm)
…the FAST.Farm output channel names/units

Added parameters for the low-res grid which is needed to generate the user-requested wind velocity output channels.
…y grids

-Added parameter support on farm%p for the low-res grid info, used for the user-requested wind velocity outputs
-Added code to attach output channel data to the AllOuts array and then write the data to an ASCII file
…ng outputs to the AllOuts array

-Fixed some bugs with the channel name search code
jjonkman and others added 15 commits May 2, 2017 14:32
Renamed phat_ce to p_ce and phat cs to p_cs.
Deleted the extra .or. test on r_tmp_plane that was in LowResGridCalcOutput, but not in HighResGridCalcOutput.
Fixed initialization of states and outputs in Wake Dynamics to match Rev. 11 plan.
Corrected several comments.
…calc in WD. Removed unused local variables. Removed < 0 check on r_tmp_plane in AWAE. changed phat_ce and phat_cs to pvec_ce and pvec_cs
This change follows Rev. 12 of the FAST.Farm plan
Updated FAST-related vs-build projects
Removed FAST.Farm related entries from RunRegistry.bat
Copy link
Collaborator

@jjonkman jjonkman left a comment

Choose a reason for hiding this comment

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

Greg,

These look OK to me. I think I've reviewed most of these changes already from the fast-farm branch.

Just FYI -- Many of the FAST-related changes in these commits are associated with FAST.Farm. I'm fine with including these in the OpenFAST dev because they won't effect FAST standalone and are not useful without the rest of the FAST.Farm routines.

@HaymanConsulting HaymanConsulting merged commit e97cd9b into OpenFAST:dev May 12, 2017
@HaymanConsulting HaymanConsulting deleted the gregpull branch May 12, 2017 14:45
@ghost ghost mentioned this pull request Apr 16, 2019
@caroledaniel caroledaniel mentioned this pull request Feb 27, 2020
@HYFOAM HYFOAM mentioned this pull request Nov 29, 2020
psakievich pushed a commit to psakievich/openfast that referenced this pull request Oct 26, 2023
Fix precision errors based on integer conversion
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.

2 participants