Skip to content

Conversation

@jniffene
Copy link
Collaborator

@jniffene jniffene commented Aug 20, 2024

Adds units to the descriptions of all the parameters written into h5 files via the Write_H5.m function.

@kmruehl kmruehl added the BEM/BEMIO related to BEMIO or BEM hydro data label Aug 21, 2024
@kmruehl kmruehl self-requested a review September 4, 2024 14:18
@kmruehl kmruehl assigned kmruehl and unassigned MShabara Sep 4, 2024
@kmruehl
Copy link
Collaborator

kmruehl commented Sep 4, 2024

@jniffene thanks for this PR that adds units to write h5. It looks like you're working off of an older version of WEC-Sim, which is why there are merge conflicts. Write_H5.m was renamed writeBEMIOH5.m, so I'm going to go ahead and pull your revisions into the latest version of WEC-Sim on dev.

@kmruehl kmruehl mentioned this pull request Sep 4, 2024
@kmruehl
Copy link
Collaborator

kmruehl commented Sep 4, 2024

@jniffene I'm going to close this PR since it's been migrated to #1306

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants