MHKiT-Software / MHKiT-Python

MHKiT-Python provides the marine renewable energy (MRE) community tools for data processing, visualization, quality control, resource assessment, and device performance.
https://mhkit-software.github.io/MHKiT/
BSD 3-Clause "New" or "Revised" License
47 stars 45 forks source link

MHKiT v0.8.1 #337

Closed ssolson closed 2 days ago

ssolson commented 4 days ago

MHKiT v0.8.1

MHKiT v0.8.1, includes bug fixes in the example notebooks and fixes the dependencies to requirements updates prior to Numpy 2.0.0.

Fixes MHKIT v0.8.0 installation issues (#334) by fixing the dependencies.

Fixes bugs in MHKiT example notebooks

ssolson commented 4 days ago

@akeeste are you around today to review this minor update?

akeeste commented 4 days ago

@ssolson yes I can review this morning. the changes are pretty straight forward since it's just 2 recent PRs. I will test a couple things locally and look into the failing test

ssolson commented 4 days ago

@ssolson yes I can review this morning. the changes are pretty straight forward since it's just 2 recent PRs. I will test a couple things locally and look into the failing test

The test failure looks like an Actions issue. I'm really hoping it just passes on a rerun. This just passed in the previous PR. We are also dropping 3.8 support soon, soo personally I'm less concerned with this test failing versus getting fresh installs of MHKiT to work for everyone ASAP.

Exact same commit where the current failure passes: https://github.com/MHKiT-Software/MHKiT-Python/actions/runs/9707092669

ssolson commented 4 days ago

Actually you can see the test passed on PR but failed on push. I need to adjust the actions in the future so it only does PR or push not both to master

image

akeeste commented 4 days ago

Actually you can see the test passed on PR but failed on push. I need to adjust the actions in the future so it only does PR or push not both to master

image

Good to know. I'm not too worried about the one-off failure then since the other instance passed. The hindcast tests are really bogging us down... I'm looking into this today

ssolson commented 2 days ago

All tests passed on re-run.