Skip to content

feat: add ltdetrv2-m-coco and ltdetrv2-l-coco checkpoint aliases#887

Merged
gabrielfruet merged 3 commits into
mainfrom
gabriel-trn-2323-upload-model-checkpoints-and-defaults-in-lightlytrain
Jul 24, 2026
Merged

feat: add ltdetrv2-m-coco and ltdetrv2-l-coco checkpoint aliases#887
gabrielfruet merged 3 commits into
mainfrom
gabriel-trn-2323-upload-model-checkpoints-and-defaults-in-lightlytrain

Conversation

@gabrielfruet

@gabrielfruet gabrielfruet commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

What has changed and why?

Registers COCO-pretrained checkpoints for the ltdetrv2-m (edgecrafter/ecvittplus-ltdetr)
and ltdetrv2-l (edgecrafter/ecvits-ltdetr) object detection models, following the
existing ltdetrv2-s-coco pattern. Adds _ECVITTPLUS_COCO_URL/_SHA256 and
_ECVITS_COCO_URL/_SHA256 constants and wires them into ModelAlias entries,
exposing four new resolvable names:

  • ltdetrv2-m-coco / edgecrafter/ecvittplus-ltdetr-coco
  • ltdetrv2-l-coco / edgecrafter/ecvits-ltdetr-coco

How has it been tested?

  • Loaded each new alias via LTDETR_MODEL_REGISTRY.get(...) and confirmed all four
    resolve to the expected config class.
  • Ran pytest tests/_task_models/ltdetr_object_detection/test_task_model.py (96 passed,
    4 skipped).

Did you update CHANGELOG.md?

  • Yes

Did you update the documentation?

  • Not needed (internal change without effects for user)

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b95ac49ca1

ℹ️ 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".

Comment thread src/lightly_train/_task_models/ltdetr_object_detection/config.py
@gabrielfruet

Copy link
Copy Markdown
Contributor Author

/codex review

@gabrielfruet

Copy link
Copy Markdown
Contributor Author

/review

@gabrielfruet
gabrielfruet marked this pull request as ready for review July 24, 2026 14:41
@gabrielfruet
gabrielfruet enabled auto-merge (squash) July 24, 2026 19:03
@gabrielfruet
gabrielfruet merged commit 30bbc9e into main Jul 24, 2026
13 checks passed
@gabrielfruet
gabrielfruet deleted the gabriel-trn-2323-upload-model-checkpoints-and-defaults-in-lightlytrain branch July 24, 2026 19:34
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.

2 participants