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

Add Ultralytics actions workflow in format.yml #71

Merged
merged 4 commits into from
Jan 1, 2024

Conversation

UltralyticsAssistant
Copy link
Member

@UltralyticsAssistant UltralyticsAssistant commented Jan 1, 2024

This pull request adds a workflow to automatically format code and documentation.

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Ultralytics introduces new automated code formatting and updates JSON to YOLO conversion scripts.

πŸ“Š Key Changes

  • πŸ†• Added a GitHub workflow (format.yml) to automatically format code in PRs and main branch pushes.
  • 🧹 Standardized quote usage in general_json2yolo.py for JSON keys from single to double quotes.
  • ✏️ Replaced manual format strings with automated formatting in various print statements.
  • πŸ—‚ Introduced a new method to delete .DS_store files on datasets.
  • πŸ›  Various code cleanups in utils.py such as standardizing image and video formats to use double quotes.

🎯 Purpose & Impact

  • The introduction of the code formatting workflow ensures consistent coding standards and improves code readability. This allows easier collaboration and code reviews.
  • Modifications to scripts ease the conversion process of JSON files to the YOLO format, benefitting users who need to transform annotated datasets efficiently for object detection tasks.
  • By cleaning and standardizing the code, the repository becomes more maintainable and accessible to new contributors or users who might not be as experienced in reading code.

@glenn-jocher glenn-jocher merged commit 67f83e6 into master Jan 1, 2024
1 check passed
@glenn-jocher glenn-jocher deleted the add-format-workflow-20240101030503 branch January 1, 2024 02:18
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.

2 participants