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

Update requirements.txt #12164

Open
wants to merge 19 commits into
base: master
Choose a base branch
from
Open

Update requirements.txt #12164

wants to merge 19 commits into from

Conversation

Powerlax
Copy link

@Powerlax Powerlax commented Sep 25, 2023

Added dill package, needed for loading custom models in detect.py

πŸ€– Generated by Copilot at c94dc81

Summary

Walkthrough

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

πŸ“Š Key Changes

  • Addition of dill>=0.3.7 to the requirements.txt file.

🎯 Purpose & Impact

  • The new requirement, dill, is likely introduced for advanced data serialization. πŸ”„
  • Users might experience improvements in data handling or features requiring serialization.
  • It ensures that users have the correct version of dill installed to avoid potential compatibility issues. βœ…

🌟 Summary

Included dill library in requirements for enhanced data serialization support. πŸ†•

Added dill package, needed for loading the model in detect.py

Signed-off-by: Vishruth Rao <116237865+Powerlax@users.noreply.github.com>
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

πŸ‘‹ Hello @Powerlax, thank you for submitting a YOLOv5 πŸš€ PR! To allow your work to be integrated as seamlessly as possible, we advise you to:

  • βœ… Verify your PR is up-to-date with ultralytics/yolov5 master branch. If your PR is behind you can update your code by clicking the 'Update branch' button or by running git pull and git merge master locally.
  • βœ… Verify all YOLOv5 Continuous Integration (CI) checks are passing.
  • βœ… Reduce changes to the absolute minimum required 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

@glenn-jocher
Copy link
Member

@Powerlax thanks for the suggestion! We appreciate your contribution to YOLOv5. Your update to requirements.txt has been duly noted and we'll make sure to review it thoroughly. Keep an eye on the repo for further updates! If you need any further assistance, feel free to reach out.

@Powerlax
Copy link
Author

Powerlax commented Nov 29, 2023

Hi @glenn-jocher, any updates on the review?

Powerlax and others added 2 commits November 28, 2023 20:54
Added the dill package for loading custom models in detect.py

Signed-off-by: Vishruth Rao <116237865+Powerlax@users.noreply.github.com>
@glenn-jocher
Copy link
Member

Thanks for your patience, @Powerlax! We're currently reviewing the changes, and we'll get back to you as soon as possible. If you have any other questions or need further assistance, feel free to let us know. Your contribution is appreciated!

Copy link
Contributor

github-actions bot commented Apr 9, 2024

All Contributors have signed the CLA. βœ…
Posted by the CLA Assistant Lite bot.

@Powerlax
Copy link
Author

I have read the CLA Document and I sign the CLA

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

3 participants