Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Change WAMIT data to mbari_snl version #168

Merged
merged 4 commits into from
Dec 6, 2023

Conversation

hamilton8415
Copy link
Collaborator

This is a simple change from one set of buoy hydrodynamic coefficients for the buoy to an updated set that was computed by the Sandia group. This new data is contained in files called mbari_snl_IR.1, mbari_snl_JR.2, mbari_snl.1, and mbari_snl_3. The WAMIT output was supplied in an .h5 file (included but not used in this repository), and converted to the .1/.3 data format by auxiliary scripts.

@hamilton8415
Copy link
Collaborator Author

The monochromatic test cases indicate that updating these hydrodynamic coefficients brings the gazebo results a bit closer to the WecSim simulator results for this system, which use the mbari_snl hydrodynamic coefficients. The comparison shown below is a particularly good example from the set of monochromatic runs (various amplitudes and periods).

Capture

Copy link
Collaborator

@andermi andermi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good! One thought would be to put the auxiliary scripts to convert he .h5 in the repo, but perhaps we can put them with our other scripts in the matlab/python processing repo we were discussing...

@hamilton8415
Copy link
Collaborator Author

Yes, I considered including that. In my typical fashion it's "results oriented", but it needs to be archived somewhere so I'll put it in and maybe add some notes to the beginning to explain how it could be improved.

@hamilton8415 hamilton8415 merged commit aa6940e into main Dec 6, 2023
2 checks passed
@hamilton8415 hamilton8415 deleted the hamilton8415/mbari_snl_WAMIT branch December 6, 2023 05:19
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants