Skip to content

fix unit test for prediction - #23

Merged
agrouaze merged 20 commits into
mainfrom
fix_issue_8
Sep 30, 2025
Merged

fix unit test for prediction#23
agrouaze merged 20 commits into
mainfrom
fix_issue_8

Conversation

@agrouaze

@agrouaze agrouaze commented Nov 25, 2024

Copy link
Copy Markdown
Member

Goal of this PR is to have a repo with working unit test (including prediction of sea state using a NN model).

  • Fix unit tests (models loaded using onnxruntime)
  • update example l2 wav file with recent product version E11
  • add try/except in the loop for production purpose.
  • improve the regexp product name parsing.
  • remove a useless unit test

@agrouaze agrouaze self-assigned this Nov 25, 2024
@agrouaze agrouaze added bug Something isn't working documentation Improvements or additions to documentation labels Nov 25, 2024
@agrouaze
agrouaze requested a review from quentinf00 May 23, 2025 11:05
@agrouaze agrouaze changed the title remove xarray-datatree dependency + small fix fix unit test for prediction ~~remove xarray-datatree dependency~~ Jun 25, 2025
@agrouaze agrouaze changed the title fix unit test for prediction ~~remove xarray-datatree dependency~~ fix unit test for prediction Jun 25, 2025
@agrouaze

Copy link
Copy Markdown
Member Author
============================================================================================= test session starts =============================================================================================
platform linux -- Python 3.12.6, pytest-8.3.3, pluggy-1.5.0
rootdir: /home1/datahome/agrouaze/sources/git/sarwaveifrproc
configfile: pyproject.toml
plugins: anyio-4.9.0, hydra-core-1.3.2, cov-6.2.1, zarr-3.1.1
collected 5 items                                                                                                                                                                                             

tests/test_output_names.py ....                                                                                                                                                                         [ 80%]
tests/test_predict.py .                                                                                                                                                                                 [100%]

============================================================================================== 5 passed in 8.74s ==============================================================================================

@agrouaze
agrouaze merged commit 0fe656d into main Sep 30, 2025
@agrouaze
agrouaze deleted the fix_issue_8 branch September 30, 2025 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants