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

VS-build updates for rc-3.5.1 and other minor updates #1818

Merged
merged 10 commits into from
Oct 19, 2023

Conversation

andrew-platt
Copy link
Collaborator

This PR is ready to merge.

Feature or improvement description
In preparation for the 3.5.1 release, a few minor issues needed addressing including the following:

  • Correct the VS project for AeroDyn_Inflow_C_Binding so that it compiles (missing files)
  • Add non-OpenMP compile options for AeroDyn, InflowWind, and AeroDyn_Inflow_C_Binding
  • Set regression test plots to full width on larger displays -- this had been set to two columns which made the plots unreadably small
  • Update the MoorDyn_Types.f90 file. Some comments had changed but it had not been regenerated.
  • Regenerate the FAST_Types.f90 file. The NodeClusterType was set as an integer in the registry file, but was a logical in the _Types file.

Related issue, if one exists

Impacted areas of the software
Build systems.

Additional supporting information

Test results, if applicable
No test results change.

RyanDavies19 and others added 4 commits October 19, 2023 11:47
Updated the MD registry and types file
- fix file name for AeroDyn_Inflow_Registry.txt in the visual studio project files
- also add `SAVE` attribute to avoid compiler warnings
Minor fixes for ADI in Visual Studio project files
@andrew-platt andrew-platt merged commit 0ac9c84 into OpenFAST:rc-3.5.1 Oct 19, 2023
19 checks passed
@andrew-platt andrew-platt mentioned this pull request Oct 19, 2023
19 tasks
@andrew-platt andrew-platt deleted the rc-3.5.1_VSprojUpdate branch October 20, 2023 16:22
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