Skip to content

Conversation

faisal-bhuiyan
Copy link
Contributor

Pull request ready to be merged.

Tests using the library were refactored in the r-tests repo.

Feature or improvement description

Major refactor of the Aerodyn/Inflowwind Python script to the C-bindings interface. This improves modularity, readability, clarity, and organization of the script.

Related issue, if one exists

N/A

Impacted areas of the software

Impacts regression tests.

Additional supporting information

N/A

Test results, if applicable

Test project /Users/fbhuiyan/dev/fb-openfast/build
    Start  81: py_ad_5MW_OC4Semi_WSt_WavesWN
1/5 Test  #81: py_ad_5MW_OC4Semi_WSt_WavesWN ....   Passed    1.04 sec
    Start  82: py_ad_B1n2_OLAF
2/5 Test  #82: py_ad_B1n2_OLAF ..................   Passed    1.06 sec
    Start 114: py_hd_5MW_OC4Semi_WSt_WavesWN
3/5 Test #114: py_hd_5MW_OC4Semi_WSt_WavesWN ....   Passed    6.25 sec
    Start 134: py_ifw_turbsimff
4/5 Test #134: py_ifw_turbsimff .................   Passed    0.32 sec
    Start 155: py_md_5MW_OC4Semi
5/5 Test #155: py_md_5MW_OC4Semi ................   Passed    2.10 sec

@faisal-bhuiyan faisal-bhuiyan self-assigned this Mar 19, 2025
@andrew-platt
Copy link
Collaborator

Thanks for revising this! It is much cleaner and more "Pythonic" now

@andrew-platt andrew-platt merged commit 79bdf79 into OpenFAST:dev-cbind Mar 27, 2025
43 of 44 checks passed
@andrew-platt andrew-platt added this to the v4.1.0 milestone Apr 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants