Skip to content

test: Create an inference test#38

Merged
dwhswenson merged 2 commits intoomsf:mainfrom
ethanholz:test/inference
Nov 11, 2024
Merged

test: Create an inference test#38
dwhswenson merged 2 commits intoomsf:mainfrom
ethanholz:test/inference

Conversation

@ethanholz
Copy link
Contributor

This pull request introduces a new GitHub Actions workflow for running inference tests using AWS instances.

Key changes include:

  • Inference Test Execution:
    • Checked out necessary scripts from the omsf-eco-infra/ci-scripts repository.
    • Run the inference.py that we checkout from the repo

@ethanholz ethanholz requested a review from dwhswenson November 7, 2024 16:33
@ethanholz ethanholz self-assigned this Nov 7, 2024
@codecov
Copy link

codecov bot commented Nov 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.73%. Comparing base (8c1b869) to head (5eb7554).
Report is 7 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #38   +/-   ##
=======================================
  Coverage   94.73%   94.73%           
=======================================
  Files           3        3           
  Lines         304      304           
=======================================
  Hits          288      288           
  Misses         16       16           
Flag Coverage Δ
unittests 94.73% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@dwhswenson dwhswenson left a comment

Choose a reason for hiding this comment

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

Looks good -- I'll trust that it will run (I assume we can't run it until it is merged into main, anyway)

@dwhswenson dwhswenson merged commit f565c52 into omsf:main Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants