Skip to content

feat: Add verified field population for VOC data loading#47

Merged
danellecline merged 2 commits intomainfrom
copilot/add-verified-field-to-voc
Feb 3, 2026
Merged

feat: Add verified field population for VOC data loading#47
danellecline merged 2 commits intomainfrom
copilot/add-verified-field-to-voc

Conversation

Copy link
Contributor

Copilot AI commented Feb 2, 2026

This PR adds the verified field to VOC data loading by modifying the parse_voc function in the tap_voc.py plugin extractor.

Changes Made

Modified mbari_aidata/plugins/extractors/tap_voc.py:

  • Added verified field with a default value of True to the object dictionary created for each bounding box parsed from VOC XML files
  • Included verified in the DataFrame columns list to ensure it's properly returned in the output

Rationale

VOC formatted data is largely verified data, so defaulting the verified field to True is appropriate for convenience. The verified field is a boolean attribute used in the Tator database for tracking annotation verification status.

Testing

  • ✅ Created and ran custom tests with both single and multiple object VOC XML files
  • ✅ Verified that the verified column is present in the DataFrame with correct True values
  • ✅ Python syntax validation passed
  • ✅ Ruff linter checks passed
  • ✅ Code review completed with no issues
  • ✅ CodeQL security scan passed with 0 alerts

The changes are minimal and focused, affecting only the data extraction in tap_voc.py without altering any other functionality.

Original prompt

This section details on the original issue you should resolve

<issue_title>feat/verified VOC label load</issue_title>
<issue_description>Currently, loads from VOC formatted data does not populate the verified field, and in largely all cases VOC data is verified so we will default to that for convenience.

This happens with the plugin extractor tap_voc.py.

verified is a boolean.</issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@danellecline
Copy link
Member

@copilot make a plan

Copy link
Contributor Author

Copilot AI commented Feb 2, 2026

@copilot make a plan

I've created a plan to add the verified field to VOC data loading. The solution involves:

  1. Modifying the parse_voc function in tap_voc.py to include a verified field set to True by default
  2. Adding it to the DataFrame columns returned by the function
  3. Testing to ensure the field is properly populated when loading VOC data

I'll implement these changes now.

@danellecline
Copy link
Member

@copilot execute the plane

Co-authored-by: danellecline <1424813+danellecline@users.noreply.github.com>
Copilot AI changed the title [WIP] Add verified field population for VOC data loading Add verified field population for VOC data loading Feb 2, 2026
Copy link
Contributor Author

Copilot AI commented Feb 2, 2026

@copilot execute the plane

The plan has been executed! The verified field has been added to VOC data loading in commit e4ff409.

Changes implemented:

  • Added 'verified': True field to the object dictionary (line 80)
  • Included verified in the DataFrame columns (line 91)

All tests passed including syntax validation, linting, code review, and security scan. The PR is ready for review.

@danellecline danellecline marked this pull request as ready for review February 3, 2026 00:13
@danellecline danellecline merged commit 38e9d4f into main Feb 3, 2026
3 checks passed
@danellecline danellecline changed the title Add verified field population for VOC data loading feat/Add verified field population for VOC data loading Feb 3, 2026
danellecline added a commit that referenced this pull request Feb 3, 2026
@danellecline danellecline changed the title feat/Add verified field population for VOC data loading feat: Add verified field population for VOC data loading Feb 3, 2026
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.

feat/verified VOC label load

2 participants