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

Update tutorial.ipynb #8507

Merged
merged 1 commit into from
Jul 7, 2022
Merged

Update tutorial.ipynb #8507

merged 1 commit into from
Jul 7, 2022

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Jul 7, 2022

Simplify TensorRT image size example

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Update to YOLOv5 tutorial for simplified TensorRT model export and inference image size specification.

📊 Key Changes

  • Changed the image size parameter in the TensorRT section of the YOLOv5 tutorial notebook from --imgsz 640 640 to --imgsz 640 for both model export and inference commands.

🎯 Purpose & Impact

  • Consistency: Ensures the image size argument is uniformly specified across the tutorial, which helps prevent confusion or errors.
  • Ease of Use: Simplifies the command for users, making it easier to understand and execute.
  • Potential Impact: Could lead to fewer user mistakes and a smoother experience for those new to YOLOv5 or TensorRT, facilitating better adoption and usability of the tutorial.

@glenn-jocher glenn-jocher self-assigned this Jul 7, 2022
@glenn-jocher glenn-jocher merged commit 36f64a9 into master Jul 7, 2022
@glenn-jocher glenn-jocher deleted the update/notebook branch July 7, 2022 10:34
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.

None yet

1 participant