-
-
Notifications
You must be signed in to change notification settings - Fork 4.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
New🌟Add ultralytics analytics dashboard (Beta) 💙 #10039
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #10039 +/- ##
==========================================
- Coverage 70.92% 70.42% -0.50%
==========================================
Files 123 124 +1
Lines 15563 15676 +113
==========================================
+ Hits 11038 11040 +2
- Misses 4525 4636 +111
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't mess around with anything custom, just tested as is on Windows and Linux. Ran well overall, a few quick comments added as suggestions.
import time | ||
|
||
import cv2 | ||
from flask import Flask, Response, jsonify, render_template |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you may want to use the check_requirements
function to do the flask
imports.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah got it, you mean as we did for solution modules?
yaml_filepath (str): dashboard configuration YAML path | ||
""" | ||
LOGGER.info("Ultralytics dashboard ... 😍") | ||
subprocess.run(["python", ROOT / "analytics/dashboard.py", yaml_filepath]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think using subprocess.run([f"{sys.executable}", ROOT / "analytics/dashboard.py", yaml_filepath])
might be a better choice. Since I have multiple python installs on my Windows PC, the current line doesn't work since it's not using my local venv
and probably not the correct version of python. The current implementation did work on Linux.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, make sense, I will update it :)
One additional idea, not sure how feasible it is, but wondering if there could be a more graceful way to terminate the example? Like adding a "Stop" or "Shutdown" example that would gracefully halt the program and cleanly exit. |
Ah, I got it @Burhan-Q Thanks for the updates :) Yes definitely your suggested changes make sense, I am working on the streamlit side, and once done I will adjust these changes in that :) |
Key Features
CLI Command for Dashboard
Python Code for Dashboard
@glenn-jocher
🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
Introducing Ultralytics Analytics Dashboard: A Real-time Visualization and Analysis Tool for Object Detection.
📊 Key Changes
🎯 Purpose & Impact
This update makes deep learning analytics more accessible and interpretable, potentially benefiting a wide range of applications from retail analytics to traffic monitoring. 🚀🔍