Skip to content

Conversation

@XevenQC
Copy link

@XevenQC XevenQC commented Jan 1, 2026

What has changed and why?

(Delete this: Please include a summary of the change and which issue is fixed. Please
also include relevant motivation and context. List any dependencies that are required
for this change.)

How has it been tested?

(Delete this: Please describe the tests that you ran to verify your changes. Provide
instructions so we can reproduce. Please also list any relevant details for your test
configuration.)

Did you update CHANGELOG.md?

  • Yes
  • Not needed (internal change)

Did you update the documentation?

  • Yes
  • Not needed (internal change without effects for user)

@CLAassistant
Copy link

CLAassistant commented Jan 1, 2026

CLA assistant check
All committers have signed the CLA.

Signed-off-by: Chao Qin <chaoqin_cmkj@163.com>
Signed-off-by: Chao Qin <chaoqin_cmkj@163.com>
@guarin
Copy link
Contributor

guarin commented Jan 6, 2026

Thanks for your PR!

Is this PR ready for review? And does the pretraining with the RTDETR model work?

@XevenQC
Copy link
Author

XevenQC commented Jan 6, 2026

Yes, it's ready and works well.

@guarin
Copy link
Contributor

guarin commented Jan 7, 2026

/review


def parse_model_name(model: str) -> tuple[str, str]:
parts = model.split("/")
parts = model.split("/", 1)
Copy link
Contributor

Choose a reason for hiding this comment

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

@XevenQC what is the motivation for this change?

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.

3 participants