Conversation
There was a problem hiding this comment.
Pull request overview
This PR promotes recent feature work to production by enabling the Reports UI in the dashboard, improving report time handling, and adding YOLO26 support across the web UI and ml-yolo service (presets, training kwargs, archive metadata, and pretrained weights).
Changes:
- Enable the Dashboard “Reports” tab and gate Reports access while a model/dashboard is running.
- Switch report date-time handling to explicit UTC semantics (UI labels + request payload formatting).
- Add YOLO26 presets and backend support (Ultralytics version bump, YOLO26 weights baked into image, YOLO26-specific config/archive handling).
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| apps/web/src/modules/ui/components/ModelRunning/ModelRunning.tsx | Adds customizable message and optional hiding of the navigation button. |
| apps/web/src/modules/reports/components/ReportsPage/ReportsPage.tsx | Enables Reports UI usage from Dashboard, adds “model running” gate using dashboard runtime status. |
| apps/web/src/modules/reports/components/CreateReportForm/CreateReportForm.tsx | Treats datetime-local input as UTC and emits UTC ISO strings; updates labels accordingly. |
| apps/web/src/modules/model/components/AdvancedSettings/presets.ts | Adds YOLO26 training presets for Ultralytics backend. |
| apps/web/src/modules/dashboard/components/DashboardPage/DashboardPage.tsx | Re-enables Reports tab and passes projectId through to ReportsPage. |
| apps/ml-yolo/requirements.txt | Bumps Ultralytics to 8.4.53 for YOLO26/MuSGD support. |
| apps/ml-yolo/Dockerfile | Adds YOLO26 pretrained weight pre-download; adjusts tools venv torch/torchvision installation. |
| apps/ml-yolo/app/ml/ultralytics_config.py | Strips dfl for YOLO26 variants and documents YOLO26 behavior. |
| apps/ml-yolo/app/ml/train_model.py | Passes model variant through to archive head patching. |
| apps/ml-yolo/app/ml/archive_patch.py | Sets YOLO head subtype based on model variant (yolo26 vs yolov8). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+54
to
+55
| --extra-index-url https://pypi.org/simple \ | ||
| torch==2.8.0+cpu torchvision==0.23.0 && \ |
Comment on lines
+31
to
+39
| const { data: cameras } = useListCamerasQuery(undefined, { | ||
| skip: !cameraApiUrl, | ||
| }); | ||
| const { cameraMxid: selectedCamera, streamName: selectedStream } = | ||
| useSelectedCameraStream(cameras); | ||
| const { isSuccess: isModelRunning } = useGetDashboardQuery( | ||
| { mxid: selectedCamera!, streamName: selectedStream! }, | ||
| { skip: !selectedCamera || !selectedStream }, | ||
| ); |
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.