refactor: ♻️ simplify FP16 input and remove redundant code#110
refactor: ♻️ simplify FP16 input and remove redundant code#110onuralpszr merged 2 commits intomainfrom
Conversation
|
👋 Hello @onuralpszr, thank you for submitting a
For more guidance, please refer to our Contributing Guide. Don't hesitate to leave a comment if you have any questions. Thank you for contributing to Ultralytics! 🚀 |
UltralyticsAssistant
left a comment
There was a problem hiding this comment.
🔍 PR Review
Made with ❤️ by Ultralytics Actions
This PR looks clean. The change is a straightforward variable rename plus removal of a redundant reassignment, and the FP16 detection logic appears unchanged.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
|
Merged! 🎉 Thanks for the thoughtful cleanup, @onuralpszr. As Leonardo da Vinci said, “Simplicity is the ultimate sophistication.” This refinement is a great example of that—clearer Really appreciate the attention to maintainability and developer clarity here. Contributions like this help keep the codebase clean and approachable for everyone. |
🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
🛠️ This PR makes a small internal cleanup in the model-loading code by simplifying how FP16 input detection is named and handled.
📊 Key Changes
model_input_fp16tofp16_inputinsrc/model.rs.model_input_fp16intofp16_input.🎯 Purpose & Impact