Skip to content

Support deployment of RFDETRKeypointPreview weights#496

Open
sergii-bond wants to merge 6 commits into
mainfrom
sb/rfdetr-keypoint-preview-deploy
Open

Support deployment of RFDETRKeypointPreview weights#496
sergii-bond wants to merge 6 commits into
mainfrom
sb/rfdetr-keypoint-preview-deploy

Conversation

@sergii-bond

@sergii-bond sergii-bond commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

Adds support to make this work:

deploy_model = RFDETRKeypointPreview.from_checkpoint(DEPLOY_CHECKPOINT_PATH)
deploy_model.deploy_to_roboflow(...)

Type of Change

  • New feature (non-breaking change that adds functionality)

Testing

  • I have tested this change locally
  • I have added/updated tests for this change

Test details:

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code where necessary, particularly in hard-to-understand areas
  • My changes generate no new warnings or errors
  • I have updated the documentation accordingly (if applicable)

Additional Context

@socket-security

socket-security Bot commented Jun 24, 2026

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedpypi/​numpy@​2.4.6 ⏵ 2.3.57510010010070

View full report

@sergii-bond sergii-bond marked this pull request as ready for review June 24, 2026 19:27
@jarbas-roboflow

Copy link
Copy Markdown

Status: ❌ Request Changes — Gist: https://gist.github.com/jarbasrf/cd3ff021bbe7076f1f49a532c64436e9. Resumo: The implementation is coherent, but it lacks a behavior-level test proving that an RF-DETR keypoint checkpoint produces a valid upload archive.

@jarbas-roboflow

Copy link
Copy Markdown

Status: ✅ Approve — Gist: https://gist.github.com/jarbasrf/d580720fed5efde64d35c0d6684adcc9. Resumo: The keypoint deployment paths are coherent and now have focused coverage for exported and raw Lightning checkpoints. No merge-blocking issue remains.

@jarbas-roboflow

Copy link
Copy Markdown

Status: ✅ Approve — Gist: https://gist.github.com/jarbasrf/57a7c83b0f08b93fb2efa372f1fedffe. Resumo: The implementation matches RF-DETR’s checkpoint and configuration contracts, with focused coverage of packaging and mismatch paths. No merge-blocking defect was found.

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.

1 participant