Add YOLO26 depth export support - #233
Conversation
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Team Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (10)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughYOLO26 depth models now have dedicated version detection, registry wiring, exporter handling, map-output NNArchive generation, model fixtures, end-to-end coverage, and updated shard manifests. ChangesYOLO26 depth conversion
Estimated code review effort: 3 (Moderate) | ~20 minutes Merge Risk: ⚪ Minimal · up to This adds YOLO26-Depth conversion and depth-map NNArchive output with matching end-to-end contract coverage. No actionable merge-readiness risk remains. Sequence Diagram(s)sequenceDiagram
participant CLI
participant VersionDetection
participant YOLO26Exporter
participant ArchiveGenerator
CLI->>VersionDetection: Detect YOLO26 DepthModel
VersionDetection-->>CLI: Return YOLOV26_DEPTH_CONVERSION
CLI->>YOLO26Exporter: Load and export depth model
YOLO26Exporter->>ArchiveGenerator: Build MapOutputParser archive
ArchiveGenerator-->>CLI: Return output_yolo26 NNArchive
Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 12 functions across 9 files. (1 skipped: 1 unsupported.) ✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
☂️ Python Coverage
Overall Coverage
New FilesNo new covered files... Modified FilesNo covered modified files...
|
Test Results 6 files 6 suites 14m 18s ⏱️ Results for commit 39802c2. |
klemen1999
left a comment
There was a problem hiding this comment.
Generally LGTM, just one small comment
Summary
Adds YOLO26-Depth support to
tools.output_yolo26.MapOutputParserwithmin_max_scaling=False.Validation
MapOutputParserNNArchive contract.0.05000.02580.14398.46 FPS56.85 FPSSummary by CodeRabbit
New Features
Tests