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 yolo.py #7061

Closed
wants to merge 1 commit into from
Closed

Update yolo.py #7061

wants to merge 1 commit into from

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Mar 20, 2022

Cleanup from #7060

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Refinement in anchor order checking within YOLOv5 model initialization.

πŸ“Š Key Changes

  • Reordered the sequence of operations in the model initialization to perform the anchor order check before calculating the network stride.

🎯 Purpose & Impact

  • 🎯 Purpose: This change ensures that anchor order verification is done earlier in the setup process, improving error detection in network configurations.
  • πŸš€ Impact: Users can expect more reliable model initialization, leading to potentially better object detection performance and stability. This small update contributes to the overall robustness of the YOLOv5 initialization process.

@glenn-jocher glenn-jocher self-assigned this Mar 20, 2022
@glenn-jocher
Copy link
Member Author

m.stride is undefined here causing error. Closing PR.

@glenn-jocher glenn-jocher deleted the update/yolo branch March 20, 2022 15:42
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