Skip to content

Move blacklisting and ICOADS-specifics from qc_suite to obs_suite#137

Merged
ludwiglierhammer merged 14 commits into
glamod:mainfrom
ludwiglierhammer:blacklisting
Jun 19, 2025
Merged

Move blacklisting and ICOADS-specifics from qc_suite to obs_suite#137
ludwiglierhammer merged 14 commits into
glamod:mainfrom
ludwiglierhammer:blacklisting

Conversation

@ludwiglierhammer

@ludwiglierhammer ludwiglierhammer commented Jun 18, 2025

Copy link
Copy Markdown
Collaborator

Pull request to issue #131.

@github-actions

Copy link
Copy Markdown

Warning
This Pull Request is coming from a fork and must be manually tagged approved
in order to perform additional testing.

@ludwiglierhammer ludwiglierhammer linked an issue Jun 18, 2025 that may be closed by this pull request
@codecov-commenter

codecov-commenter commented Jun 18, 2025

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 94.57364% with 7 lines in your changes missing coverage. Please review.

Project coverage is 16.97%. Comparing base (4438cf3) to head (b100b93).
Report is 15 commits behind head on main.

Files with missing lines Patch % Lines
...arine_processing/obs_suite/modules/blacklisting.py 92.42% 5 Missing ⚠️
...mod_marine_processing/obs_suite/scripts/level1a.py 94.11% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #137      +/-   ##
==========================================
+ Coverage   15.53%   16.97%   +1.44%     
==========================================
  Files          39       42       +3     
  Lines        6934     7063     +129     
==========================================
+ Hits         1077     1199     +122     
- Misses       5857     5864       +7     
Flag Coverage Δ
unittests 16.97% <94.57%> (+1.44%) ⬆️

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.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ludwiglierhammer ludwiglierhammer merged commit ac33910 into glamod:main Jun 19, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

Move blacklisting from qc_suite to where?

2 participants