Skip to content

Update Pascal VOC export API usage in Grounding DINO notebook#444

Draft
rkrisa wants to merge 1 commit into
roboflow:mainfrom
rkrisa:codex/update-pascal-voc-export-api
Draft

Update Pascal VOC export API usage in Grounding DINO notebook#444
rkrisa wants to merge 1 commit into
roboflow:mainfrom
rkrisa:codex/update-pascal-voc-export-api

Conversation

@rkrisa

@rkrisa rkrisa commented Jun 20, 2026

Copy link
Copy Markdown

Summary

  • Replaces the removed sv.detections_to_voc_xml(...) call with the current Pascal VOC export API.
  • Passes image_shape=image.shape, matching the current detections_to_pascal_voc signature.
  • Cleans up the now-unused depth variable in the export cell.

Validation

  • Parsed
    otebooks/automated-dataset-annotation-and-evaluation-with-grounding-dino.ipynb as JSON.
  • Installed supervision==0.29.0.post0 into a temporary target directory and confirmed sv.detections_to_voc_xml is no longer available.
  • Verified sv.dataset.formats.pascal_voc.detections_to_pascal_voc(...) exists with the expected image_shape parameter.
  • Ran a small local export smoke test that generated Pascal VOC XML from a synthetic sv.Detections object.

Fixes #236.

@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@CLAassistant

CLAassistant commented Jun 20, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

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.

Incompatibility with latest supervision version: no detections_to_voc_xml

2 participants