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

Release 3.5.2 #1949

Merged
merged 23 commits into from
Jan 18, 2024
Merged

Release 3.5.2 #1949

merged 23 commits into from
Jan 18, 2024

Conversation

andrew-platt
Copy link
Collaborator

@andrew-platt andrew-platt commented Dec 19, 2023

Feature or improvement description
Pull request to merge rc-3.5.2 into main and create a tagged release for v3.5.2.

See the milestone and project pages for additional information

https://github.com/OpenFAST/openfast/milestone/12

Test results, if applicable
See GitHub Actions

Release checklist:

  • Update the documentation version in docs/conf.py
  • Update the versions in docs/source/user/api_change.rst
  • Verify readthedocs builds correctly
  • Create a tag in OpenFAST
  • Create a merge commit in r-test and add a corresponding tag
  • Compile executables for Windows builds
    • FAST_SFunc.mexw64
    • OpenFAST-Simulink_x64.dll
    • openfast_x64.exe
    • DISCON.dll (x64)
    • AeroDyn_Driver
    • AeroDyn_Inflow_C_Binding
    • BeamDyn_Driver
    • HydroDyn_Driver
    • HydroDyn_C_Binding (x64)
    • InflowWind_Driver
    • IfW_C_Binding (x64)
    • MoorDyn_Driver
    • FAST.Farm (x64)

Changelog

General

Build systems

#1948 Pass Python_EXECUTABLE to pfunit, add error check on Python version

Module changes

AeroDyn

#1913 ADI: memory leak in ADI_UpdateStates

AWAE

#1963 FAST.Farm, Mod_AmbWind=3: add error if HR grid not centered on turbine in Y dimension

HydroDyn

#1872 Fix segfault in HD when no outputs specified

Regression tests

#1886 Update floating MHK case input files

Input file changes

No input files change with this release, as this only includes minor bugfixes.

Full list of changes: https://openfast.readthedocs.io/en/main/source/user/api_change.html

Full input file sets: https://github.com/OpenFAST/r-test/tree/v3.5.2 (example input files from the regression testing)

deslaughter and others added 12 commits November 13, 2023 11:11
This commit fixes a bug where HydroDyn would segfault if no outputs were
specified in the input file.
Fix segfault in HD when no outputs specified
Update BlCenBt inputs in MHK r-tests
The ADI_UpdateStates routine makes a copy of the AD_Input, but never destroyed that data after usage.  This could lead to memory continously getting allocated during a simulation with either ADI_C_Binding or the AD driver.
ADI: probably memory leak in ADI_UpdateStates
This allows pfunit to use the same interpreter found by the main OF cmake.

NOTE: pfunit uses `PYTHON_EXECUTABLE`
Pass Python_EXECUTABLE from main OF cmake to external pfunit cmake
@andrew-platt andrew-platt marked this pull request as ready for review December 19, 2023 21:35
Option to turn off unit_tests from cmake
@andrew-platt
Copy link
Collaborator Author

andrew-platt commented Jan 9, 2024

@andrew-platt andrew-platt merged commit 4b6337f into main Jan 18, 2024
38 checks passed