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

Handling of Paths for SrvD UserSubs Input Files #2079

Merged
merged 1 commit into from
Mar 8, 2024

Conversation

andrew-platt
Copy link
Collaborator

@andrew-platt andrew-platt commented Mar 8, 2024

Feature or improvement description

In Issue #2059, @rdamiani pointed out that the existing method for handling file paths with UserSubs is difficult as it requires putting all the user input files in the same location as the ServoDyn file. This PR gives a method that from @rdamiani for a method he has had good success with.

This PR is entirely @rdamini's work merely copied into a PR by me.

Co-authored-by:rdamiani r.damiani@RRDengineering.com

Related issue, if one exists
Closes #2059

Impacted areas of the software
ServoDyn controls where the UserSubs option is used (PCMode, VSContrl, HSSBrMode, or YCMode =3).

Additional supporting information
The usage of the affected routines is a very specialized case that we don't directly support, but certainly want to make life easier for users of these features.

Test results, if applicable
None.

@andrew-platt andrew-platt added this to the v3.5.3 milestone Mar 8, 2024
@andrew-platt andrew-platt requested a review from deslaughter March 8, 2024 00:31
@andrew-platt andrew-platt self-assigned this Mar 8, 2024
@andrew-platt andrew-platt force-pushed the f/SrvD_UserSubs_path branch from ebe865e to f2ab5bc Compare March 8, 2024 00:32
@rdamiani pointed out in issue OpenFAST#2059 that the existing method for handling file paths with `UserSubs` is difficult as it requires putting all the user input files in the same location as the ServoDyn file.  This commit is from recomendations from @rdamiani for a method he has had good success with.

Co-authored-by:rdamiani <r.damiani@RRDengineering.com>
@andrew-platt andrew-platt merged commit bbcf464 into OpenFAST:rc-3.5.3 Mar 8, 2024
19 checks passed
@andrew-platt andrew-platt deleted the f/SrvD_UserSubs_path branch March 8, 2024 16:49
@@ -439,6 +439,7 @@ typedef ^ ParameterType IntKi StCCmode - - - "Structural control control mode {0
typedef ^ ParameterType IntKi NumOuts - - - "Number of parameters in the output list (number of outputs requested)" -
typedef ^ ParameterType IntKi NumOuts_DLL - - - "Number of logging channels output from the DLL (set at initialization)" -
typedef ^ ParameterType CHARACTER(1024) RootName - - - "RootName for writing output files" -
typedef ^ ParameterType CHARACTER(1024) PriPath - - - "Path of the primary SD input file " -
Copy link
Contributor

Choose a reason for hiding this comment

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

SD is SubDyn, so this should be changed to SrvD.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Corrected directly on rc-3.5.3 with commit 7a2c43e

andrew-platt added a commit to andrew-platt/openfast that referenced this pull request Mar 11, 2024
@andrew-platt andrew-platt mentioned this pull request Mar 26, 2024
27 tasks
andrew-platt added a commit that referenced this pull request Apr 11, 2024
Summary of all PR's included in this merge (PR #2121):

* Add 4 linearization regression tests
* Upgrade to setup-python@v5 and cache@v4 for GH actions
* Update fast_linearization_file.py to use less memory
* Use env var to set CTest parallel runs, reduce to 2
* Use more efficient method to compare linearization matrices in regression tests.
* Fix printing bug in Python Linear Regression test script
* GH actions: upgrade upload-artifact to v4
* Fix bug in HD_Perturb_x where SS_Exctn%numStates was used instead of SS_Rdtn%numStates
* GH actions: upgrade checkout to v4 and setup-python to v5
* Remove typo in warning, retains c/con option for output flags for backwards compatibility
* Bathymetry grid bug and slope normal vector bug fix
* Change input string size for bathgrid and waterkin
* Lin tests: increase out precision, error printing full filename
* Add SubDyn Linearization test case
* Lin: incorrect BD index in dUdy for AD motion
* Lin: sum BD blade root motion to hub moment m_ud term (consistent with documentation)
* Fix bug in BD linearization resulting from reference rotation change
* Improve handling of Paths for SrvD UserSubs Input Files
* Revert " Resolve BeamDyn initial strain for rotated blade"
* BeamDyn: Disable changing the reference frame
* BeamDyn: Implement new initial strain fix based on OpenTurbine documentation
* SrvD: fix typo in registry from PR #2079
* BeamDyn: Restore previous fix to pitch actuator logic in SetParameters
* FASTlib: Turbine(:) array must use Fortran index 1 start
* FASTlib: change ExternInitData%TurbineID to %TurbIDforName
* FASTlib: change iTurb inputs to routines with c-bindings to index 0
* Set 5MW_Land_DLL_WTurb_cpp as a CPP_API test, not CPP_DRIVER
* Adjust if statements in FAST_OpFM_Step for iTurb being in range 1:NumTurbines.
* CPP testing: fix logic setting exec name in CMakeList.txt
* TurbSim: increase filename to 1024 characters
* VSbuild: always build Registry as release x64
* Add initialization of MHK, WtrDpth, MSL2SWL in IfW_InitInp for AWAE module
* Use of potentially unallocated array in AeroDyn_AllBldNdOuts.
* VSbuild: increase stack size for FAST.Farm
* CMake: openmp optional for FF and CPP if not explicitly requested (#2120)
* Update docs for release 3.5.3
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.

4 participants