Skip to content
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

Rename 'labels' to 'instances' #9066

Merged
merged 4 commits into from
Aug 21, 2022
Merged

Rename 'labels' to 'instances' #9066

merged 4 commits into from
Aug 21, 2022

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Aug 21, 2022

Align naming with HUB

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Improved logging format for training and validation in YOLOv5.

πŸ“Š Key Changes

  • Updated printed logger format in train.py, changing the names and spacing for better clarity (e.g., from gpu_mem to GPU_mem, box to box_loss, etc.).
  • Adjusted the progress bar description format to match the new logger changes.
  • In val.py, updated result logging format (e.g., from Labels to Instances) and increased the class name field width for better alignment.

🎯 Purpose & Impact

  • Enhanced Readability: The new format provides clearer and more descriptive logging information, aiding users in quickly understanding training and validation outputs.
  • Consistency in Metrics: Consistent naming such as *_loss for loss metrics and Instances helps in understanding the entries as part of a standard format across different sections.
  • Better Logging UX: Increasing the width for class names prevents text from wrapping, improving the overall user experience during model progress checks.

These changes mainly impact the user interface in terms of readability and consistency, with no direct impact on model performance or functionality. Users can now enjoy a more intuitive understanding of what's happening during model training and validation. πŸš€

@glenn-jocher glenn-jocher self-assigned this Aug 21, 2022
@glenn-jocher
Copy link
Member Author

@AyushExel @robotwhispering what do you think?

Existing printout

Screenshot 2022-08-21 at 15 21 21

Proposed PR printout

Screenshot 2022-08-21 at 15 19 41

@AyushExel
Copy link
Contributor

I prefer proposed printout.

@glenn-jocher glenn-jocher merged commit 0b8639a into master Aug 21, 2022
@glenn-jocher glenn-jocher deleted the rename/instances branch August 21, 2022 13:50
@glenn-jocher
Copy link
Member Author

@AyushExel @robotwhispering PR is merged. Thank you for your contributions to YOLOv5 πŸš€ and Vision AI ⭐

ctjanuhowski pushed a commit to ctjanuhowski/yolov5 that referenced this pull request Sep 8, 2022
* Rename labels to instances

* Rename labels to instances

* align val
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants