Skip to content

Video: GStreamer subsystem overhaul + audit fixes#14499

Open
HTRamsey wants to merge 4 commits into
mavlink:masterfrom
HTRamsey:video/gst-source-factory
Open

Video: GStreamer subsystem overhaul + audit fixes#14499
HTRamsey wants to merge 4 commits into
mavlink:masterfrom
HTRamsey:video/gst-source-factory

updoots

955b3f5
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Jun 27, 2026 in 0s

30.44% (target 30.00%)

View this Pull Request on Codecov

30.44% (target 30.00%)

Details

Codecov Report

❌ Patch coverage is 18.57765% with 1122 lines in your changes missing coverage. Please review.
✅ Project coverage is 30.44%. Comparing base (f29efd3) to head (955b3f5).
⚠️ Report is 113 commits behind head on master.

Files with missing lines Patch % Lines
...anager/VideoReceiver/GStreamer/GstVideoReceiver.cc 0.00% 253 Missing ⚠️
...anager/VideoReceiver/GStreamer/GstSourceFactory.cc 31.09% 108 Missing and 138 partials ⚠️
.../VideoManager/VideoReceiver/GStreamer/GStreamer.cc 12.12% 116 Missing and 29 partials ⚠️
...er/VideoReceiver/GStreamer/GStreamerEnvironment.cc 10.89% 121 Missing and 18 partials ⚠️
src/Utilities/Platform/GraphicsSetup.cc 0.00% 85 Missing ⚠️
src/VideoManager/VideoManager.cc 17.34% 64 Missing and 17 partials ⚠️
...anager/VideoReceiver/GStreamer/GStreamerHelpers.cc 39.06% 43 Missing and 35 partials ⚠️
...anager/VideoReceiver/GStreamer/GStreamerLogging.cc 26.82% 35 Missing and 25 partials ⚠️
src/Settings/VideoSettings.cc 44.82% 2 Missing and 14 partials ⚠️
...Manager/VideoReceiver/GStreamer/GstVideoReceiver.h 0.00% 6 Missing ⚠️
... and 5 more
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #14499      +/-   ##
==========================================
+ Coverage   25.47%   30.44%   +4.97%     
==========================================
  Files         769      786      +17     
  Lines       65912    66746     +834     
  Branches    30495    30914     +419     
==========================================
+ Hits        16788    20323    +3535     
+ Misses      37285    32532    -4753     
- Partials    11839    13891    +2052     
Files with missing lines Coverage Δ
src/LogManager/LogManager.h 100.00% <ø> (ø)
src/Settings/VideoSettings.h 100.00% <ø> (+85.71%) ⬆️
src/VideoManager/VideoManager.h 25.00% <ø> (+25.00%) ⬆️
...Streamer/HwBuffers/common/GstHwFrameTexturesBase.h 14.28% <ø> (ø)
...iver/GStreamer/HwBuffers/common/GstHwVideoBuffer.h 0.00% <ø> (ø)
...VideoReceiver/GStreamer/QGCQVideoSinkController.cc 33.08% <ø> (ø)
...ideoReceiver/GStreamer/gstqgc/GstQgcVideoFormats.h 66.66% <ø> (ø)
.../VideoReceiver/GStreamer/gstqgc/gstqgcqvideosink.h 100.00% <ø> (ø)
...ideoReceiver/GStreamer/gstqgc/gstqgcvideosinkbin.h 100.00% <ø> (ø)
...er/VideoReceiver/Offscreen/QGCOffscreenRenderer.cc 0.00% <ø> (ø)
... and 22 more

... and 403 files with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a8a6cc4...955b3f5. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.