Skip to content

Conversation

RizwanMunawar
Copy link
Member

@RizwanMunawar RizwanMunawar commented Aug 5, 2025

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

The pyproject.toml file has been removed from the repository, simplifying the project structure.

πŸ“Š Key Changes

  • Deleted the pyproject.toml file, which previously managed build, packaging, and dependency settings for a template Python package.

🎯 Purpose & Impact

  • 🧹 Simplifies the repository: Removes unused or unnecessary configuration files, making the project easier to maintain.
  • πŸš€ Reduces confusion: Prevents users from mistakenly thinking this repository is intended for packaging or distribution.
  • πŸ“¦ No impact on core functionality: This change does not affect the main notebooks or user workflows.

Overall, this update streamlines the repository and clarifies its purpose for all users.

Signed-off-by: Muhammad Rizwan Munawar <[email protected]>
@UltralyticsAssistant UltralyticsAssistant added dependencies Dependencies and packages enhancement New feature or request labels Aug 5, 2025
@RizwanMunawar
Copy link
Member Author

RizwanMunawar commented Aug 5, 2025

@lakshanthad @onuralpszr, I am removing pyproject.toml as I don't think it's necessary for the notebooks repository.

@UltralyticsAssistant
Copy link
Member

πŸ‘‹ Hello @RizwanMunawar, thank you for submitting an ultralytics/notebooks πŸš€ PR! To ensure a seamless integration of your work, please review the following checklist:

  • βœ… Define a Purpose: Clearly explain the purpose of your fix or feature in your PR description, and link to any relevant issues. Ensure your commit messages are clear, concise, and adhere to the project's conventions.
  • βœ… Synchronize with Source: Confirm your PR is synchronized with the ultralytics/notebooks main branch. If it's behind, update it by clicking the 'Update branch' button or by running git pull and git merge main locally.
  • βœ… Ensure CI Checks Pass: Verify all Ultralytics Continuous Integration (CI) checks are passing. If any checks fail, please address the issues.
  • βœ… Update Documentation: Update the relevant documentation for any new or modified features.
  • βœ… Add Tests: If applicable, include or update tests to cover your changes, and confirm that all tests are passing.
  • βœ… Sign the CLA: Please ensure you have signed our Contributor License Agreement if this is your first Ultralytics PR by writing "I have read the CLA Document and I sign the CLA" in a new message.
  • βœ… Minimize Changes: Limit your changes to the minimum necessary for your bug fix or feature addition. "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 more guidance, please refer to our Contributing Guide. Don’t hesitate to leave a comment if you have any questions. Thank you for contributing to Ultralytics! ✨

This is an automated response; an Ultralytics engineer will review your PR and assist you soon.

@onuralpszr
Copy link
Member

onuralpszr commented Aug 5, 2025

@lakshanthad @onuralpszr, I am removing pyproject.toml as I don't think it's necessary for the notebooks repository.

For formatters and other custom scripts or we might even need for docs maybe πŸ€”

@lakshanthad
Copy link
Member

lakshanthad commented Aug 5, 2025

@RizwanMunawar In my opinion, at the current stage, since notebooks repository only contains iPython notebooks, pyproject.toml looks redundant. However as @onuralpszr highlighted, if you have any other plans in future such as building docs out of this repo or do more stuff, then we might need it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Dependencies and packages enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants