-
Notifications
You must be signed in to change notification settings - Fork 184
Aqwa-phase-bug-resolved #855
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
Conversation
|
Thanks @salhus ! |
|
Thanks for merging it @kmruehl ! |
|
Dear Salhus, the phase of excitation force read by this program is not consistent with the value in LIS file. Could you please help test whether the program needs to be improved? Thank you. |
|
@tuisong1106 |
I used AQWA to calculate the interaction between two cylinders, and the reading program found that the excitation force was not matched, and the added mass and radiation Damping were opposite in the upper right corner (the influence of 1 on 2) and the lower left corner (the influence of 2 on 1). |
|
@tuisong1106 Cheers, |
|
@tuisong1106 and @salhus can you open a new issue and move this discussion to it? thank you. |
|
Thanks @salhus |
|
Hi @tuisong1106 , |
Added a line to add a phase offset to the phase data extracted from aqwa to account for the different sign conventions used.
Thus follows from #846