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

Merged
merged 2 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

Automated formatting workflow addition and code refactoring in plots.py and various utility files.

📊 Key Changes

  • Added a .github/workflows/format.yml to implement automatic code formatting.
  • In plots.py, refactored plotting code for readability and consistency, involving quotation mark standardization and decomposition of complex function calls.
  • Updated utility files (KLT.py, MSV.py, NLS.py, etc.) to use clearer numerical representations and consistent string quoting.
  • Improved variable naming and string quoting consistency in images.py.
  • Fixed a minor path string issue in vid2images.py.

🎯 Purpose & Impact

  • The new GitHub Actions workflow will ensure consistent code style across commits and pull requests.
  • Refactored code will enhance readability and maintainability for developers.
  • Users should experience no direct impact, as these changes are mostly internal code quality improvements without altering functionality.

@glenn-jocher glenn-jocher changed the title Add format workflow Add Ultralytics actions workflow in format.yml Jan 1, 2024
@glenn-jocher glenn-jocher merged commit 5007293 into master Jan 1, 2024
@glenn-jocher glenn-jocher deleted the add-format-workflow-20240101025027 branch January 1, 2024 01:55
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.

None yet

2 participants