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

TorchScript single-output fix #7261

Merged
merged 1 commit into from
Apr 3, 2022
Merged

TorchScript single-output fix #7261

merged 1 commit into from
Apr 3, 2022

Conversation

glenn-jocher
Copy link
Member

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

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Refined export options and inference flow.

πŸ“Š Key Changes

  • Improved readability of the export formats list in export.py.
  • Distinguished between PyTorch (pt) and TorchScript (jit) model inference flows in common.py.

🎯 Purpose & Impact

  • Enhanced code clarity in export options supports easier maintenance and updates. πŸ› οΈ
  • Separating inference handling provides cleaner code structure and allows for potential individual optimizations for PyTorch and TorchScript backends. πŸš€
  • These updates should not affect end users directly, but improved codebase health can lead to fewer bugs and better future features. 🌱

@glenn-jocher glenn-jocher self-assigned this Apr 3, 2022
@glenn-jocher glenn-jocher merged commit 8bc839e into master Apr 3, 2022
@glenn-jocher glenn-jocher deleted the fix/ts branch April 3, 2022 20:51
BjarneKuehl pushed a commit to fhkiel-mlaip/yolov5 that referenced this pull request Aug 26, 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.

1 participant