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

Ultralytics Code Refactor https://ultralytics.com/actions #14

Merged
merged 1 commit into from
Jun 21, 2024

Conversation

glenn-jocher
Copy link
Contributor

This Ultralytics PR refactors code to improve performance and readability. 🔄

Key changes include:

  • 🚀 Optimized various functions for faster execution.
  • 🧩 Simplified complex logic for better understanding and maintenance.
  • 🗑️ Removed redundant code to streamline operations.
  • 📚 Improved code structure and organization.

These changes aim to enhance the overall quality and efficiency of the code. 🌟

Learn more about Ultralytics:

Summary by Sourcery

This pull request refactors the codebase to optimize performance, simplify logic, remove redundant code, and improve overall code structure. It also updates CI workflows for better readability and enhances test cases with improved documentation.

  • Enhancements:
    • Optimized various functions for faster execution and improved performance.
    • Simplified complex logic to enhance readability and maintainability.
    • Removed redundant code to streamline operations and reduce clutter.
    • Improved code structure and organization for better clarity.
  • CI:
    • Updated CI workflow configuration for consistency and readability.
  • Tests:
    • Enhanced test cases with additional docstrings for better understanding of test purposes.
    • Refactored test directory path creation to use f-strings for consistency.

* Refactor code for speed and clarity

* Auto-format by https://ultralytics.com/actions

* Update onnxslim/core/optimizer.py

Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>

* Auto-format by https://ultralytics.com/actions

* Update graph.py

---------

Co-authored-by: UltralyticsAssistant <[email protected]>
Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>
@inisis inisis merged commit f01c295 into inisis:main Jun 21, 2024
2 checks passed
@inisis
Copy link
Owner

inisis commented Jun 21, 2024

Thanks for your contribution!

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