-
-
Notifications
You must be signed in to change notification settings - Fork 16.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PR draft of new YOLO and COCO data set loaders #4527
Conversation
…ata_set_loaders # Conflicts: # utils/datasets/cache.py
@glenn-jocher yes sir! I noticed and merged everything into my PR. |
@SkalskiP was scanning our TODO list for more low hanging fruit. I merged item 19 EarlyStopping today in #4576, and saw item 6. native n-channel image support (i.e. 1-ch greyscale, 4-ch or n-ch hyperspectral images), which seems easy enough, but would require updates throughout datasets.py and augmentations.py, which you've got your own updates to now I see. There's also 4. auto batch-size, and 13. Annotator class I can do. EDIT: 13. Annotator class is now complete and merged in #4591 |
# Conflicts: # detect.py # models/common.py
@glenn-jocher I read your comment today. I already merged all changes that you have done over the weekend. As for the next steps, it would be great if you would hold back with |
@SkalskiP yeah that's what I figured. No worries, there's plenty more on the TODO list that doesn't interact with datasets.py, I'll tackle those first. |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions YOLOv5 🚀 and Vision AI ⭐. |
🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
This PR involves a major refactoring of the dataset utilities used in Ultralytics' YOLOv5, which includes separating utility functions into newly created files and preparing for future development with new tests.
📊 Key Changes
utils/datasets.py
toutils/datasets_old.py
and updated all imports to reflect this change.tests/__init__.py
,tests/utils/__init__.py
,tests/utils/datasets/__init__.py
,tests/utils/datasets/test_coco.py
,tests/utils/test_file.py
,tests/utils/test_utils.py
) to ensure code quality and correctness.pytest
torequirements.txt
to enable the new testing framework.utils/datasets/coco.py
andutils/datasets/yolo.py
.utils/datasets/core.py
that includes dataset loaders likeCOCODataset
andYOLODataset
.utils/datasets/image_cache.py
andutils/datasets/label_cache.py
) to improve performance when working with large datasets.utils/datasets/todo.txt
indicating future work and improvements to be made in the datasets module.🎯 Purpose & Impact
todo.txt
suggests there is ongoing work to improve the datasets utilities further, giving insight into the direction of future updates.