Skip to content

Add Instance Segmentation Transforms#836

Merged
yutong-xiang-97 merged 6 commits into
mainfrom
yutong-trn-2276-add-instance-segmentation-transforms
Jul 8, 2026
Merged

Add Instance Segmentation Transforms#836
yutong-xiang-97 merged 6 commits into
mainfrom
yutong-trn-2276-add-instance-segmentation-transforms

Conversation

@yutong-xiang-97

@yutong-xiang-97 yutong-xiang-97 commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

What has changed and why?

This PR stacks on top of #835.

It adds LT-DETR instance segmentation transform support on top of the object detection transform refactor:

  • add instance segmentation dataset mask handling
  • add LT-DETR instance segmentation transform wiring
  • add a reusable ltdetr_transforms instance segmentation transform module
  • add focused transform coverage for the new instance segmentation path

How has it been tested?

Unit tests.

Did you update CHANGELOG.md?

  • Yes
  • Not needed (internal change)

Did you update the documentation?

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

@yutong-xiang-97 yutong-xiang-97 changed the title Add instance segmentation transforms Add Instance Segmentation Transforms Jul 3, 2026
@yutong-xiang-97
yutong-xiang-97 marked this pull request as ready for review July 3, 2026 16:18
@yutong-xiang-97

Copy link
Copy Markdown
Contributor Author

/review

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b7caf1ca53

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/lightly_train/_transforms/ltdetr_transforms/instance_segmentation.py Outdated
Comment thread src/lightly_train/_transforms/ltdetr_transforms/instance_segmentation.py Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 653459f581

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/lightly_train/_transforms/ltdetr_transforms/instance_segmentation.py Outdated
@yutong-xiang-97
yutong-xiang-97 force-pushed the yutong-trn-2271-refactor-ltdetr-object-detection-transforms branch from 51848b5 to 4bef13d Compare July 7, 2026 15:05
@yutong-xiang-97
yutong-xiang-97 force-pushed the yutong-trn-2276-add-instance-segmentation-transforms branch from 1ed8999 to 6de2792 Compare July 7, 2026 16:12

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6de2792838

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Base automatically changed from yutong-trn-2271-refactor-ltdetr-object-detection-transforms to main July 7, 2026 17:52
@yutong-xiang-97
yutong-xiang-97 force-pushed the yutong-trn-2276-add-instance-segmentation-transforms branch from ab12869 to 3437116 Compare July 8, 2026 08:34

@gabrielfruet gabrielfruet left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

In general LGTM, I just have some questions

Comment thread src/lightly_train/_transforms/ltdetr_transforms/utils.py
Comment thread src/lightly_train/_commands/train_task.py
@yutong-xiang-97
yutong-xiang-97 enabled auto-merge (squash) July 8, 2026 17:05
@yutong-xiang-97
yutong-xiang-97 merged commit 08ce9f5 into main Jul 8, 2026
13 checks passed
@yutong-xiang-97
yutong-xiang-97 deleted the yutong-trn-2276-add-instance-segmentation-transforms branch July 8, 2026 17:31
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