Skip to content

Add tests for notebooks #56

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

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

Add tests for notebooks #56

wants to merge 5 commits into from

Conversation

RizwanMunawar
Copy link
Member

@RizwanMunawar RizwanMunawar commented Jun 4, 2025

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Added automated testing for a YOLO notebook to ensure reliability and smooth updates. πŸš€πŸ““

πŸ“Š Key Changes

  • Introduced a GitHub Actions workflow to automatically run and test a specific YOLO notebook.
  • Workflow triggers on pushes, pull requests to main, and daily at 08:00 UTC.
  • Sets up a Python 3.11 environment and installs necessary Jupyter tools.
  • Executes the "how-to-train-ultralytics-yolo-on-medical-pills-dataset" notebook as a test.

🎯 Purpose & Impact

  • Ensures the notebook runs correctly with every code change or update, catching errors early. πŸ›‘οΈ
  • Improves reliability and user confidence in the provided notebook examples.
  • Streamlines development by automating testing, saving time for contributors and maintainers. ⏱️

@UltralyticsAssistant UltralyticsAssistant added detect Object Detection issues, PR's devops GitHub Devops or MLops labels Jun 4, 2025
@UltralyticsAssistant
Copy link
Member

πŸ‘‹ Hello @RizwanMunawar, thank you for submitting a PR to ultralytics/notebooks πŸš€! This is an automated response to help guide your contributionβ€”an Ultralytics engineer will review and assist you soon.

Please take a moment to review the following checklist to help ensure a smooth review process:

  • βœ… Define a Purpose: Clearly state the purpose of your PR in the description, and link any relevant issues. Ensure your commit messages are clear and follow project conventions.
  • βœ… Synchronize with Source: Make sure your PR is up to date with the main branch of ultralytics/notebooks. If not, use the 'Update branch' button or git pull and git merge main locally.
  • βœ… Ensure CI Checks Pass: Confirm that all Ultralytics Continuous Integration (CI) checks are passing. Please resolve any CI failures.
  • βœ… Update Documentation: Update the relevant documentation if your PR adds or modifies features.
  • βœ… Add Tests: If applicable, ensure tests are included or updated for your changes, and all tests are passing.
  • βœ… Sign the CLA: If this is your first PR to Ultralytics, please sign our Contributor License Agreement (CLA) by commenting "I have read the CLA Document and I sign the CLA".
  • βœ… Minimize Changes: Limit your changes to the minimum necessary for your fix or feature. "It is not daily increase but daily decrease, hack away the unessential. The closer to the source, the less wastage there is." β€” Bruce Lee

For further details, see our Contributing Guide. If you have questions, feel free to leave a comment. Thank you for helping improve Ultralytics! πŸš€βœ¨

@RizwanMunawar RizwanMunawar self-assigned this Jun 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
detect Object Detection issues, PR's devops GitHub Devops or MLops
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants