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

Colorstr() updates #1909

Merged
merged 6 commits into from
Jan 12, 2021
Merged

Colorstr() updates #1909

merged 6 commits into from
Jan 12, 2021

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Jan 12, 2021

This PR implements minor colorstr() updates and bug fixes.

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Simplified color string usage and improved logging and dataset handling in YOLOv5.

📊 Key Changes

  • Refactored colorstr function to default to blue and bold text if no color arguments are provided.
  • Enhanced training and testing data loaders with the prefix argument for better log readability.
  • Simplified wandb import and log messages across training scripts.
  • Used f-strings for cleaner and more consistent error and logging messages.
  • Modified autoanchor utilities to use the updated colorstr.
  • Included the prefix argument when caching labels for clarity in dataset preparation logs.
  • Adjusted plot range and tick intervals in study comparison plots.

🎯 Purpose & Impact

  • 📈 Improved Clarity: Streamline color usage and prefix logging to make output more understandable for users.
  • 🚀 Enhanced Readability: Using f-strings and consistent log prefacing improves code readability and error tracking.
  • 🛠️ Refactoring: Code quality is enhanced with function simplifications and removal of redundant code.
  • 📌 User Experience: Adjustments to visual plots allow for better interpretation of model comparison studies.

@glenn-jocher glenn-jocher merged commit dd03b20 into master Jan 12, 2021
@glenn-jocher glenn-jocher deleted the wandb branch January 12, 2021 18:33
KMint1819 pushed a commit to KMint1819/yolov5 that referenced this pull request May 12, 2021
* W&B ImportError message fix

* colorstr() updates

* colorstr() updates

* colorstr() default to 'blue', 'bold'

* train: magenta

* train: blue
taicaile pushed a commit to taicaile/yolov5 that referenced this pull request Oct 12, 2021
* W&B ImportError message fix

* colorstr() updates

* colorstr() updates

* colorstr() default to 'blue', 'bold'

* train: magenta

* train: blue
BjarneKuehl pushed a commit to fhkiel-mlaip/yolov5 that referenced this pull request Aug 26, 2022
* W&B ImportError message fix

* colorstr() updates

* colorstr() updates

* colorstr() default to 'blue', 'bold'

* train: magenta

* train: blue
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.

1 participant