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

Linear damping for rod elements in MoorDyn #2342

Merged
merged 7 commits into from
Jul 23, 2024
Merged

Conversation

RBergua
Copy link
Contributor

@RBergua RBergua commented Jul 22, 2024

I believe this PR is ready to be merged.

Feature or improvement description
This is a development done by Front Energies (Johyun Kyoung) as part of the USFLOWT project. The system uses large buoyant cans modeled as rods in MoorDyn.
image

Previous to this development, it was only possible to include viscous damping from drag forces (quadratic).

The proposed code allows to include transverse linear damping per unit of length [N/(m/s)/m] for the rod element.

The updated section of the rod element in the input file looks as follows:
image

Impacted areas of the software
MoorDyn

@andrew-platt
Copy link
Collaborator

Is the LinDamp column in the input file optional, or required?

@andrew-platt andrew-platt self-assigned this Jul 22, 2024
@andrew-platt andrew-platt added this to the v4.0.0 milestone Jul 22, 2024
@RBergua
Copy link
Contributor Author

RBergua commented Jul 22, 2024

If the user doesn't define the LinDamp column, the code assumes that there is no damping. It should be backcompatible.

Screenshot from the debugging that we were using in the development:
image

Transverse linear damping per unit of length [N/(m/s)/m] is added to the rod element
RBergua and others added 3 commits July 22, 2024 13:24
- remove non-standard format identifier `(F)` on internal read, and add comment on what it does
- a few more tab characters
- a couple stray dos line returns
@andrew-platt andrew-platt merged commit 4b8eba6 into OpenFAST:dev Jul 23, 2024
21 checks passed
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.

2 participants