Skip to content

Dat 637 make datamint upload accepts csv files as class mapping - #51

Merged
Lucashsmello merged 3 commits into
mainfrom
DAT-637-make-datamint-upload-accepts-csv-files-as-class-mapping
Jul 25, 2025
Merged

Dat 637 make datamint upload accepts csv files as class mapping#51
Lucashsmello merged 3 commits into
mainfrom
DAT-637-make-datamint-upload-accepts-csv-files-as-class-mapping

Conversation

@Lucashsmello

Copy link
Copy Markdown
Collaborator

No description provided.

@Lucashsmello
Lucashsmello requested a review from Copilot July 25, 2025 14:31

Copilot AI 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.

Pull Request Overview

This PR extends the datamint upload tool to accept CSV files as class mapping in addition to YAML files. The changes enable users to provide segmentation class mappings through CSV files with columns for index, RGB values, and names.

  • Adds support for CSV format in segmentation class mapping files
  • Implements CSV parsing logic with pandas for metadata extraction
  • Updates function signatures and improves metadata file handling

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

File Description
datamint/client_cmd_tools/datamint_upload.py Adds CSV parsing functionality and improves metadata handling for segmentation files
datamint/apihandler/root_api_handler.py Adds null check for segmentation files to prevent processing errors

Comment thread datamint/client_cmd_tools/datamint_upload.py Outdated
Comment thread datamint/client_cmd_tools/datamint_upload.py Outdated
Comment thread datamint/client_cmd_tools/datamint_upload.py Outdated
Comment thread datamint/client_cmd_tools/datamint_upload.py
Comment thread datamint/client_cmd_tools/datamint_upload.py
@Lucashsmello
Lucashsmello merged commit 48fd52d into main Jul 25, 2025
3 checks passed
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