Skip to content
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

Remove tensorrt pip install check #7439

Merged
merged 1 commit into from
Apr 15, 2022
Merged

Remove tensorrt pip install check #7439

merged 1 commit into from
Apr 15, 2022

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Apr 15, 2022

Resolves #7431

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Streamlined import process for NVIDIA TensorRT in export script.

πŸ“Š Key Changes

  • Removed check_requirements function for TensorRT import in export.py.
  • Direct import of TensorRT library now without the prior version check.

🎯 Purpose & Impact

  • Simplification: The change makes the codebase cleaner by eliminating the version check before importing TensorRT.
  • Ease of use: Users can now expect a more straightforward import process for TensorRT, which is used for optimizing deep learning models.
  • Potential Performance Improvement: As the development moves forward, the YOLOv5 model can leverage the latest TensorRT optimizations without version restrictions, potentially improving performance for end-users. πŸš€
  • Update Readiness: Encourages users to maintain the latest version of TensorRT, ensuring compatibility with new features and optimizations provided by Ultralytics.

@glenn-jocher glenn-jocher self-assigned this Apr 15, 2022
@glenn-jocher glenn-jocher merged commit 3eefab1 into master Apr 15, 2022
@glenn-jocher glenn-jocher deleted the update/trt branch April 15, 2022 19:48
@glenn-jocher glenn-jocher mentioned this pull request Apr 15, 2022
1 task
BjarneKuehl pushed a commit to fhkiel-mlaip/yolov5 that referenced this pull request Aug 26, 2022
ctjanuhowski pushed a commit to ctjanuhowski/yolov5 that referenced this pull request Sep 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TensorRT bindings version
1 participant