Skip to content

Conversation

kprokofi
Copy link
Contributor

πŸ“ Description

resolves #1312, #1316

✨ Type of Change

Select the type of change your PR introduces:

  • 🐞 Bug fix – Non-breaking change which fixes an issue
  • πŸš€ New feature – Non-breaking change which adds functionality
  • πŸ”¨ Refactor – Non-breaking change which refactors the code base
  • πŸ’₯ Breaking change – Changes that break existing functionality
  • πŸ“š Documentation update
  • πŸ”’ Security update
  • πŸ§ͺ Tests

πŸ§ͺ Testing Scenarios

Describe how the changes were tested and how reviewers can test them too:

  • βœ… Tested manually
  • πŸ€– Run automated end-to-end tests

βœ… Checklist

Before submitting the PR, ensure the following:

  • πŸ” PR title is clear and meaningful
  • ✍️ PR description clearly explains the changes and their reason
  • πŸ“ I have linked the PR to the corresponding GitHub Issues, if any
  • πŸ’¬ I have commented my code, especially in hard-to-understand areas
  • πŸ“„ I have made corresponding changes to the documentation
  • βœ… I have added tests that prove my fix is effective or my feature works

@Copilot Copilot AI review requested due to automatic review settings September 18, 2025 14:16
@kprokofi kprokofi requested a review from a team as a code owner September 18, 2025 14:16
Copy link
Contributor

@Copilot Copilot AI left a comment

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 updates OTX version and fixes DETR augmentations by disabling problematic data augmentation configurations across multiple RT-DETR model manifests.

  • Adds dataset_preparation configuration with disabled iou_random_crop for RT-DETR models
  • Disables color_jitter augmentation for DFINE-X model
  • Standardizes augmentation settings across detection models to resolve training issues

Reviewed Changes

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

File Description
rtdetr_50.yaml Adds dataset preparation config with disabled iou_random_crop
rtdetr_18.yaml Adds dataset preparation config with disabled iou_random_crop
rtdetr_101.yaml Adds dataset preparation config with disabled iou_random_crop
dfine_x.yaml Disables color_jitter augmentation

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@kprokofi kprokofi requested a review from Copilot September 18, 2025 14:28
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 4 out of 5 changed files in this pull request and generated no new comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] training Detection model RTDetr-R101, RTDetr-50, RTDetr-R18 fails everytime
1 participant