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

Use new Ultralytics Retry action #2301

Merged
merged 1 commit into from
Nov 8, 2024
Merged

Conversation

pderrenger
Copy link
Member

@pderrenger pderrenger commented Nov 8, 2024

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:

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

A GitHub workflow file adjustment reduces the number of retries for link checking operations.

📊 Key Changes

  • Reduced retry attempts from 3 to 2 in GitHub Actions workflow for link validation using Lychee tool.

🎯 Purpose & Impact

  • Efficiency Improvement: Fewer retries could streamline workflow runs, saving time and resources. ⏱️
  • Resource Management: Minimizes redundant network requests, which can optimize the workflow's resource usage. 🌐

@UltralyticsAssistant UltralyticsAssistant added the devops GitHub Devops or MLops label Nov 8, 2024
@UltralyticsAssistant
Copy link
Member

👋 Hello @pderrenger, thank you for your contribution to the ultralytics/yolov3 repository! This is an automated response, and an Ultralytics engineer will assist you soon. To help streamline the review process, please follow this checklist:

  • Define a Purpose: Kindly ensure your pull request description clearly explains the intention of your changes. If your PR addresses specific issues, please link to any relevant issues.
  • Synchronize with Source: Check if your branch is up to date with the ultralytics/yolov3 main branch. Update it if necessary by using the 'Update branch' button or running git pull and git merge main.
  • Ensure CI Checks Pass: Confirm that all Ultralytics Continuous Integration (CI) checks are passing. Address any failures so your PR can be merged smoothly.
  • Update Documentation: If your changes impact the behavior of the software, update the documentation to reflect these modifications.
  • Add Tests: Ensure that new or modified features are covered by tests and that all tests pass successfully.
  • Sign the CLA: Ensure you have signed our Contributor License Agreement if this is your first contribution to Ultralytics by commenting "I have read the CLA Document and I sign the CLA."
  • Minimize Changes: Focus your changes solely on the bug fix or feature addition without introducing unnecessary modifications. As Bruce Lee said, "It is not daily increase but daily decrease—hack away the unessential."

If your PR relates to a bug and does not yet include a minimum reproducible example (MRE), please add one. This assists in efficiently diagnosing and resolving issues.

For additional guidance, please check our Contributing Guide. Thank you for enhancing the YOLOv3 project! 🚀

@pderrenger pderrenger merged commit e4da77f into master Nov 8, 2024
6 checks passed
@pderrenger pderrenger deleted the refactor-20241108123327 branch November 8, 2024 11:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devops GitHub Devops or MLops
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants