Skip to content

Conversation

@MShabara
Copy link
Contributor

Updates the Matlab and the Ubuntu versions in the workflow.

Limits the tests to ubuntu-22.04 rather than ubuntu-latest (ubuntu-24.04). Only Matlab 2024x is compatible with ubuntu-24.04.

Noting that the Ubuntu 20.04 runner image will be fully unsupported by github on April 1, 2025

@MShabara MShabara requested review from kmruehl and salhus February 17, 2025 15:51
@salhus
Copy link
Contributor

salhus commented Feb 17, 2025

Hi @MShabara,

This seems to be a good solution to maintain compatibility. This can be merged.

@MShabara MShabara merged commit 2d0d19b into WEC-Sim:dev Feb 17, 2025
12 checks passed
MShabara added a commit to jtgrasb/WEC-Sim that referenced this pull request Feb 17, 2025
* Updates the Ubuntu and Matlab versions
MShabara added a commit that referenced this pull request Feb 17, 2025
* Fix irregular wave elevation with multiple directions

* Updates the Ubuntu and Matlab versions (#1420)

* Updates the Ubuntu and Matlab versions

---------

Co-authored-by: Mohamed Shabara <84589678+MShabara@users.noreply.github.com>
@kmruehl kmruehl added SCM source code mangagement and warnings Tests/CI related WEC-Sim tests or Continuous Integration labels Feb 19, 2025
kmruehl added a commit that referenced this pull request Mar 1, 2025
* fix missing mask initialization line for multiple wave headings;

* update moordyn readme from WEC-Sim/MoorDyn repo

* Updates the Ubuntu and Matlab versions (#1420)

* Updates the Ubuntu and Matlab versions

* reverting to last four MATLAB releases

---------

Co-authored-by: Mohamed Shabara <84589678+MShabara@users.noreply.github.com>
Co-authored-by: Ruehl <kmruehl@sandia.gov>
akeeste added a commit that referenced this pull request Mar 25, 2025
* updated the intro

* fixes a bug with the QTF interpolation function (#1396)

* fix #1288, cleanup waveclass (main) (#1417)

* fix on pDis function call

* preliminary fix for issue #1288

* Update readAQWA.m (#1253)

* body mask fixes for #1346

* add warning when variable hydro turned on for drag or nonhydro bodies

* flex body - add missing mask lines, define h5 button callback

* fix #1288 cleanup waveClass

* adding bodyClass, should match

---------

Co-authored-by: jtgrasb <87095491+jtgrasb@users.noreply.github.com>
Co-authored-by: akeeste <akeeste@sandia.gov>

* Fix irregular wave elevation with multiple directions (#1403)

* Fix irregular wave elevation with multiple directions

* Updates the Ubuntu and Matlab versions (#1420)

* Updates the Ubuntu and Matlab versions

---------

Co-authored-by: Mohamed Shabara <84589678+MShabara@users.noreply.github.com>

* new body class constructor, no need for h5file

* removed disp used in debugging

* short check to ensure hydro struct provided is the right hydro struct

* cleaned up one comment

* rename hydrodata to hydroinput in body constructor

* move input checking to body.checkInputs()

* make body.hydroStruct a private variable

* make body.hydroStruct public and alphabetize

* clear body.hydroStruct once used

* brief documentation

* function to build body class hydro format struct from BEMIO format

* update hydro to hydroData function for other features

* fix bug in reading water depth, ex.irf.t and dofs

* make readH5ToStruct function identically to writeBEMIOH5 for GBM modes

* read GBM data and irf times correctly

* reference function in documentation

* update function docstring

---------

Co-authored-by: Shabara <mshabara@nrel.gov>
Co-authored-by: Mohamed Shabara <84589678+MShabara@users.noreply.github.com>
Co-authored-by: dforbush2 <dforbus@sandia.gov>
Co-authored-by: jtgrasb <87095491+jtgrasb@users.noreply.github.com>
Co-authored-by: akeeste <akeeste@sandia.gov>
MShabara added a commit that referenced this pull request Mar 28, 2025
* updated the intro

* fixes a bug with the QTF interpolation function (#1396)

* fix #1288, cleanup waveclass (main) (#1417)

* fix on pDis function call

* preliminary fix for issue #1288

* Update readAQWA.m (#1253)

* body mask fixes for #1346

* add warning when variable hydro turned on for drag or nonhydro bodies

* flex body - add missing mask lines, define h5 button callback

* fix #1288 cleanup waveClass

* adding bodyClass, should match

---------

Co-authored-by: jtgrasb <87095491+jtgrasb@users.noreply.github.com>
Co-authored-by: akeeste <akeeste@sandia.gov>

* Fix irregular wave elevation with multiple directions (#1403)

* Fix irregular wave elevation with multiple directions

* Updates the Ubuntu and Matlab versions (#1420)

* Updates the Ubuntu and Matlab versions

---------

Co-authored-by: Mohamed Shabara <84589678+MShabara@users.noreply.github.com>

* Enhances function flexibility

* Fix warnings

* change coercion for consistency with documentation

---------

Co-authored-by: Shabara <mshabara@nrel.gov>
Co-authored-by: Mohamed Shabara <84589678+MShabara@users.noreply.github.com>
Co-authored-by: dforbush2 <dforbus@sandia.gov>
Co-authored-by: jtgrasb <87095491+jtgrasb@users.noreply.github.com>
Co-authored-by: akeeste <akeeste@sandia.gov>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

SCM source code mangagement and warnings Tests/CI related WEC-Sim tests or Continuous Integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants