Skip to content

[Backport release/0.0.14] mark a test as self_hosted - #4192

Merged
Dreamsorcerer merged 2 commits into
release/0.0.14from
backport/4190-to-release/0.0.14
Sep 18, 2026
Merged

Dreamsorcerer merged 2 commits into
release/0.0.14from
backport/4190-to-release/0.0.14

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Description

Backport of #4190 to release/0.0.14.

Co-authored-by: bogwi <bogdan@dimensional.org>
(cherry picked from commit 416fc30)
@greptile-apps

greptile-apps Bot commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

Safe to merge: the updated marker selects the detection test in its intended self-hosted environment.

What we checked:

  • T-Rex executed the marker contract harness to validate the contract against HEAD^ and HEAD with CI=1, using the Linux self-hosted marker expression. T-Rex
  • The harness’s before-run state showed 1 deselected in 0.10s with exit code 5. T-Rex
  • The harness’s after-run state showed 1 passed in 0.10s with exit code 0. T-Rex
  • A blocker prevented completing the full fixture by failing to install the test environment due to portaudio.h being absent, with pyaudio==0.2.14 pulled via unitree-webrtc-connect. T-Rex

Summary

  • Reclassifies test_detection2d so the Linux self-hosted test selection includes it under CI conditions.
  • The marker comparison confirmed that the previous test was deselected while the updated test runs and passes with the same self-hosted selection expression.
  • No issues requiring changes were found.

Reviews (1) · Last reviewed commit: "mark a test as self_hosted (#4190)"

@codecov

codecov Bot commented Sep 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

@@                Coverage Diff                @@
##             release/0.0.14    #4192   +/-   ##
=================================================
  Coverage                  ?   79.02%           
=================================================
  Files                     ?     1471           
  Lines                     ?   139760           
  Branches                  ?    12015           
=================================================
  Hits                      ?   110447           
  Misses                    ?    25890           
  Partials                  ?     3423           
Components Coverage Δ
Tests 94.81% <0.00%> (?)
Flag Coverage Δ
OS-ubuntu-24.04-arm 74.97% <100.00%> (?)
OS-ubuntu-latest 75.67% <100.00%> (?)
Py-3.10 75.44% <100.00%> (?)
Py-3.11 75.65% <100.00%> (?)
Py-3.12 75.65% <100.00%> (?)
SelfHosted-Large 30.48% <100.00%> (?)
SelfHosted-Linux 36.60% <100.00%> (?)

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

Files with missing lines Coverage Δ
...perception/detection/type/detection2d/test_bbox.py 100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions github-actions Bot added the ready-to-merge Required CI checks have passed on this PR label Sep 18, 2026
@Dreamsorcerer
Dreamsorcerer merged commit 39beead into release/0.0.14 Sep 18, 2026
29 of 30 checks passed
@Dreamsorcerer
Dreamsorcerer deleted the backport/4190-to-release/0.0.14 branch September 18, 2026 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge Required CI checks have passed on this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants