feat: 🚀 add max_det in InferenceConfig and add it as a CLI argument.
#58
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
Adds a
--max-detCLI option and standardizes the config/API naming frommax_detectionstomax_detfor consistent detection limits 🧰✅📊 Key Changes
InferenceConfigfield and builder method:max_detections→max_detwith_max_detections(...)→with_max_det(...)🔁--max-det(default300) to control the maximum number of detections returned per image 🖥️🎛️--max-detinto prediction config creation so it affects runtime outputs ⚙️README.md, crate docs) and all unit/integration tests to match the new naming 📚🧪config.max_detwhen limiting kept detections 🧩🎯 Purpose & Impact
max_det), reducing confusion and aligning CLI + Rust API 🧠✨with_max_detectionsormax_detectionsmust be updated to the new names300), so most users won’t see changes unless they rely on the old API name 🔒✅