Skip to content

Conversation

@nathanmtom
Copy link

This is an update to reflect changes to the NEMOH input and output files that necessitate an update to readNEMOH.

Changes are:

  • The NEMOH.cal allows for three options to set wave angular frequency, frequency, or period. An additional column with 1-3 to represent these three choices was added compared to last release.
  • ExcitationForce.tec has changed the name of the header from 'Diffraction force' to 'Excitation force'

The files were updated to be compatible with v3.0.0 release and existing NEMOH files in the WEC-Sim Examples BEMIO folder.

Note: As it appears NEMOH is providing updates the WEC-Sim team may decide in the future to make readNEMOH only compatible with the latest NEMOH release.
Note: The Examples BEMIO folder need to be updated as changes to the NEMOH.cal and input.txt were made such that the existing input files are incompatible. Placing a project board ticket to update.

@nathanmtom nathanmtom added the BEM/BEMIO related to BEMIO or BEM hydro data label Jul 28, 2023
@kmruehl kmruehl added the Bug bug in WEC-Sim source, high priority label Aug 9, 2023
@kmruehl kmruehl requested review from kmruehl and salhus August 9, 2023 14:45
@kmruehl
Copy link
Collaborator

kmruehl commented Aug 10, 2023

@nathanmtom should this go into master or dev?

@kmruehl kmruehl removed the request for review from salhus August 11, 2023 19:51
@kmruehl kmruehl self-assigned this Aug 11, 2023
Copy link
Collaborator

@kmruehl kmruehl left a comment

Choose a reason for hiding this comment

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

Thanks @nathanmtom! I tested this on NEMOH v3.0, and with the current BEMIO examples. It's ready for a merge. Also, thanks for adding updating the BEMIO examples to NEMOH v3.0 to the project board.

@kmruehl kmruehl merged commit 7d537d2 into WEC-Sim:dev Aug 11, 2023
@kmruehl
Copy link
Collaborator

kmruehl commented Aug 11, 2023

Oops! @nathanmtom should this go in dev or master?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BEM/BEMIO related to BEMIO or BEM hydro data Bug bug in WEC-Sim source, high priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants