docs: add ltdetrv2 m/l COCO benchmarks and checkpoints#900
docs: add ltdetrv2 m/l COCO benchmarks and checkpoints#900gabrielfruet wants to merge 4 commits into
Conversation
|
/codex review |
|
/review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ee4cf61790
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| | picodet-s-coco | 26.7\* | 2.2\* | 1.17 | 416×416 | | ||
| | picodet-l-coco | 32.0\* | 2.4\* | 3.75 | 416×416 | | ||
| | **ltdetrv2-s-coco** | **50.7** | **5.4** | **9.9** | **640×640** | | ||
| | **ltdetrv2-m-coco** | **53.1** | **7.95** | **21.1** | **640×640** | |
There was a problem hiding this comment.
Same comments for README. I would add the plot here with some comparision and also move picodet to the very end.
There was a problem hiding this comment.
Or alternatively, group the results with model backbones. I think up to this point the table is a bit too long to read.
There was a problem hiding this comment.
Then I'll wait for the plot
Depends on: #887 (must merge first or together — the m/l
-cocoaliases don't exist until then)What has changed and why?
Docs previously only covered
ltdetrv2-s. This adds the newltdetrv2-m/lCOCO benchmark results and checkpoint aliases to
object_detection.md, matchingthe numbers already landing in the README (#894) and the checkpoint wiring in
#887.
How has it been tested?
Docs-only change; visually checked the rendered markdown table for alignment.
Did you update CHANGELOG.md?
Did you update the documentation?