-
-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
OnnxSlim support #12681
Comments
👋 Hello @inisis, thank you for your interest in Ultralytics YOLOv8 🚀! We recommend a visit to the Docs for new users where you can find many Python and CLI usage examples and where many of the most common questions may already be answered. If this is a 🐛 Bug Report, please provide a minimum reproducible example to help us debug it. If this is a custom training ❓ Question, please provide as much information as possible, including dataset image examples and training logs, and verify you are following our Tips for Best Training Results. Join the vibrant Ultralytics Discord 🎧 community for real-time conversations and collaborations. This platform offers a perfect space to inquire, showcase your work, and connect with fellow Ultralytics users. InstallPip install the pip install ultralytics EnvironmentsYOLOv8 may be run in any of the following up-to-date verified environments (with all dependencies including CUDA/CUDNN, Python and PyTorch preinstalled):
StatusIf this badge is green, all Ultralytics CI tests are currently passing. CI tests verify correct operation of all YOLOv8 Modes and Tasks on macOS, Windows, and Ubuntu every 24 hours and on every commit. |
@glenn-jocher Can you take a look, I have tested on yolov8n model, and it works really good for reducing number of operators. |
@inisis does OnnxSlim also speed up for TensorRT (since model have to be convert to onnx first) ? |
I do believe so, onnxslim depends on nv onnx-graphsurgeon, but have better perfromance. |
does it works with onnx-simplify? |
@inisis I will make a pull request to apply this in the export function |
@inisis pull request for the integration, thanks for the tool :) |
@Kayzwer Hi, CI failed, but I think it's not a problem with onnxslim, and how can I compare the performance with/without onnxslim, and one thing to mention, onnxslim can be used in arm devices. |
|
make sure to rename the output onnx model and run the functions separately |
@Kayzwer thank you! But I think I have seen the benchmark performance before, and it's generated by CI. |
@glenn-jocher can you please review it |
@inisis Thanks for the heads up! The PR is on Glenn's radar, and he'll review it as soon as possible. Thanks for your contribution! 🚀 |
thanks LLM |
Search before asking
Description
Hi, we have developed a tool called onnxslim, which can help slim exported onnx model, and it's pure python, and it works well on yolo, should we intergrate it into awesome ultralytics.
Use case
when export to onnx, we can use onnxslim to slim our exported model
Additional
No response
Are you willing to submit a PR?
The text was updated successfully, but these errors were encountered: