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 #73

Closed

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

Introduced a new GitHub workflow for automatic code formatting and updated JSON to YOLO conversion scripts.

📊 Key Changes

  • Added .github/workflows/format.yml for auto-formatting code in PRs and pushes to main and master branches.
  • Updated string quotation consistency in general_json2yolo.py by replacing single quotes with double quotes.
  • Standardized numerical formatting by ensuring decimals are consistent (e.g., 0. to 0.0) for clarity.
  • Enhanced printing statements for better readability and consistency.
  • Modified labelbox_json2yolo.py to include improvements similar to general_json2yolo.py.

🎯 Purpose & Impact

  • The new GitHub workflow ensures code adheres to Ultralytics formatting standards, which helps maintain code quality and readability.
  • Consistent quotation and number formatting create a more uniform codebase, making it easier to read and maintain.
  • Clearer print statements aid in debugging and provide more informative messages for users.
  • Overall, these changes should enhance both developer experience and the end-user's ability to utilize conversion scripts effectively.

@glenn-jocher glenn-jocher deleted the add-format-workflow-20240101031715 branch January 1, 2024 02:17
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