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

AeroDyn14 Driver #1

Closed
sayerhs opened this issue Jan 6, 2017 · 6 comments
Closed

AeroDyn14 Driver #1

sayerhs opened this issue Jan 6, 2017 · 6 comments

Comments

@sayerhs
Copy link
Contributor

sayerhs commented Jan 6, 2017

AeroDyn14 driver contains USE AeroDyn_Types instead of USE AeroDyn14_Types. Looks like the code needs to be updated to reflect the changes since introduction of AeroDyn v15.

sayerhs referenced this issue in sayerhs/OpenFAST Apr 5, 2017
jjonkman referenced this issue in jjonkman/OpenFAST Jun 15, 2017
Updating fast-farm to compile with cmake
@rafmudaf
Copy link
Collaborator

For future reference, I've updated cmake to include aerodyn14_driver in the build and I've updated AD_Driver to use the aerodyn14 types and function calls. However, it looks like updates need to be made to the registry

Scanning dependencies of target aerodyn14_driver
[ 95%] Building Fortran object modules-local/aerodyn14/CMakeFiles/aerodyn14_driver.dir/src/AD_Driver.f90.o
/Users/rmudafor/Development/openfast/modules-local/aerodyn14/src/AD_Driver.f90:68:23:

    InitInData%ADOptions%ADFile = 'MyInputFileName.inp'
                       1
Error: 'adoptions' at (1) is not a member of the 'ad14_initinputtype' structure
/Users/rmudafor/Development/openfast/modules-local/aerodyn14/src/AD_Driver.f90:77:67:

    CALL AD14_Init( InitInData, u(1), p,  x, xd, z, OtherState, y, m, TimeInterval, InitOutData, ErrStat, ErrMsg )
                                                                   1
Error: Symbol 'm' at (1) has no IMPLICIT type
/Users/rmudafor/Development/openfast/modules-local/aerodyn14/src/AD_Driver.f90:106:85:

       CALL AD14_CalcOutput( Time, u(1), p, x, xd, z, OtherState, y, ErrStat, ErrMsg )
                                                                                     1
Error: Type mismatch in argument 'm' at (1); passed INTEGER(4) to TYPE(ad14_miscvartype)
/Users/rmudafor/Development/openfast/modules-local/aerodyn14/src/AD_Driver.f90:115:84:

       call AD14_UpdateStates( Time, u(1), p, x, xd, z, OtherState, ErrStat, ErrMsg )
                                                                                    1
Error: Type mismatch in argument 'n' at (1); passed TYPE(ad14_inputtype) to INTEGER(4)
/Users/rmudafor/Development/openfast/modules-local/aerodyn14/src/AD_Driver.f90:127:69:

    CALL AD14_End( u(1), p, x, xd, z, OtherState, y, ErrStat, ErrMsg )
                                                                     1
Error: Type mismatch in argument 'm' at (1); passed INTEGER(4) to TYPE(ad14_miscvartype)

This is available at https://github.com/rafmudaf/openfast/tree/bugfix/aerodyn14_driver

@bjonkman
Copy link
Contributor

I would be surprised if anyone ever used AD14's driver. I'm pretty sure it was just there to help John compile when he converted AD13 to the FAST modularization framework. It doesn't do anything useful does it? Other than verify that AD14 can compile and possibly run without seg faults?

ADOptions is probably left over from AD13.

@rafmudaf
Copy link
Collaborator

@bjonkman no, it doesn't do much. It just calls the AD14 init, calc output and update states over three time steps, and the end method. It doesn't actually read an input file.

In fact, looking at this driver in more detail I'm wondering was it ever actually used to drive a working AD14? It looks if John used it in development he stopped pretty early as the InitInData variable which is passed to AD14_Init isn't actually initialized at all.

@bjonkman
Copy link
Contributor

I personally would just delete the AD14 driver. AD14 is supposed to be phased out anyway, right?

@rafmudaf
Copy link
Collaborator

@bjonkman I completely agree.

@michaelasprague @jjonkman @ghaymanNREL any objection to this? I've created a pull request for this at #92.

On the question of phasing out AD14, I think that deserves its own issue.

@rafmudaf
Copy link
Collaborator

ps new issue created for ad14 phase out #93

ebranlard referenced this issue in ebranlard/openfast Mar 8, 2019
Initializing NumSC2CtrlGlob. Was uninitialized earlier.
andrew-platt pushed a commit that referenced this issue Jul 31, 2020
Thanks for doing that! It works now.
andrew-platt pushed a commit that referenced this issue Jul 31, 2020
andrew-platt pushed a commit that referenced this issue Jul 31, 2020
rafmudaf added a commit that referenced this issue Apr 21, 2021
* Update rules for Intel OneAPI compiler detection

CMake 3.20 differentiates between older Intel compilers and the Intel
OneAPI compilers (which is now detected as `IntelLLVM`). This commit
introduces the necessary changes to CMake logic to allow building using
OneAPI compiler stack with the latest CMake version.

* Update CMake check for Intel compiler (#1)

Co-authored-by: Rafael M Mudafort <rafael.mudafort@nrel.gov>
andrew-platt pushed a commit that referenced this issue Oct 27, 2021
Move environment vars for HD init into InitInpType
dzalkind pushed a commit to dzalkind/openfast that referenced this issue Jun 22, 2022
openfast(66108,0x10fb1edc0) malloc: *** error for object 0x7fe4a223a000: pointer being freed was not allocated
openfast(66108,0x10fb1edc0) malloc: *** set a breakpoint in malloc_error_break to debug

Program received signal SIGABRT: Process abort signal.

Backtrace for this error:
\#0  0x103a3f33d
\OpenFAST#1  0x103a3e76d
\OpenFAST#2  0x7fff68c225fc
Abort trap: 6
dzalkind pushed a commit to dzalkind/openfast that referenced this issue Jun 22, 2022
dzalkind pushed a commit to dzalkind/openfast that referenced this issue Jun 22, 2022
deslaughter added a commit that referenced this issue Feb 28, 2023
More updates for new `DbKi=R8Ki` behavior
andrew-platt pushed a commit that referenced this issue Apr 4, 2023
andrew-platt pushed a commit that referenced this issue Oct 2, 2023
Update regression tests, and merge in latest rc-3.5.1
andrew-platt pushed a commit that referenced this issue Apr 4, 2024
Automatically build OpenFAST docker images
andrew-platt pushed a commit that referenced this issue Jul 23, 2024
MD: minor cleanup on Lin Damp feature
andrew-platt pushed a commit that referenced this issue Aug 16, 2024
@hhh850 hhh850 mentioned this issue Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants