refactor(aug): rename aug_config to aug_configs across project - #1103
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR renames the augmentation preset module reference from rfdetr.datasets.aug_config to rfdetr.datasets.aug_configs across runtime code, tests, and user-facing docs/notebooks to match the new module name and avoid import errors.
Changes:
- Updated Python imports in training/data modules and dataset code to use
rfdetr.datasets.aug_configs. - Updated tests to import augmentation presets/configs from
aug_configs. - Updated documentation and example notebooks to reference
aug_configsin code examples.
Reviewed changes
Copilot reviewed 9 out of 10 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| tests/training/test_module_data.py | Updates test import to new augmentation module name. |
| tests/datasets/test_kornia_transforms.py | Updates augmentation preset imports to aug_configs. |
| tests/datasets/test_augmentations.py | Updates augmentation config/preset imports to aug_configs. |
| src/rfdetr/training/module_data.py | Switches default augmentation config import to aug_configs. |
| src/rfdetr/datasets/coco.py | Switches default augmentation config import to aug_configs (but some docstring references still mention aug_config). |
| src/rfdetr/datasets/aug_configs.py | Adds/updates the new augmentation presets module (docstring still contains outdated import/example formatting). |
| notebooks/release-demo_1-5.ipynb | Updates example imports to aug_configs. |
| docs/learn/train/augmentations.md | Updates docs examples to import from aug_configs. |
| docs/learn/train/advanced.md | Updates code block import to aug_configs (but one inline reference still mentions aug_config). |
| docs/cookbooks/release-demo_1-5.ipynb | Updates example imports to aug_configs. |
Codecov Report✅ All modified and coverable lines are covered by tests. ❌ Your project check has failed because the head coverage (81%) is below the target coverage (95%). You can increase the head coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## develop #1103 +/- ##
=======================================
Coverage 81% 81%
=======================================
Files 106 106
Lines 11046 11046
=======================================
+ Hits 8955 8956 +1
+ Misses 2091 2090 -1 🚀 New features to boost your workflow:
|
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request updates all references to the
aug_configmodule throughout the codebase and documentation, changing them to the new module nameaug_configs. This ensures consistency and prevents import errors after the module was renamed. The changes affect both code files and documentation/tutorials.Codebase updates:
rfdetr.datasets.aug_configsinstead ofrfdetr.datasets.aug_configfor augmentation presets and configs. [1] [2] [3] [4] [5]Documentation and notebooks:
aug_configsinstead ofaug_config. [1] [2] [3] [4] [5]