-
-
Notifications
You must be signed in to change notification settings - Fork 16.7k
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
Gardio integration with hugging face spaces #8784
Conversation
Updates the requirements on [protobuf](https://github.com/protocolbuffers/protobuf) to permit the latest version. - [Release notes](https://github.com/protocolbuffers/protobuf/releases) - [Changelog](https://github.com/protocolbuffers/protobuf/blob/main/generate_changelog.py) - [Commits](https://github.com/protocolbuffers/protobuf/commits) --- updated-dependencies: - dependency-name: protobuf dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]>
Update protobuf requirement from <=3.20.1 to <4.21.5
for more information, see https://pre-commit.ci
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👋 Hello @xiang-wuu, 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 upstream/master. If your PR is behind upstream/master an automatic GitHub Actions merge may be attempted by writing /rebase in a new comment, or by running the following code, replacing 'feature' with the name of your local branch:
git remote add upstream https://github.com/ultralytics/yolov5.git
git fetch upstream
# git checkout feature # <--- replace 'feature' with local branch name
git merge upstream/master
git push -u origin -f
- ✅ Verify all 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
pre-commit.ci autofix |
for more information, see https://pre-commit.ci
…with autopep8 formatting to fix pre-commit.ci test
for more information, see https://pre-commit.ci
@glenn-jocher can you review this PR. |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions YOLOv5 🚀 and Vision AI ⭐. |
👋 Hello there! We wanted to let you know that we've decided to close this pull request due to inactivity. We appreciate the effort you put into contributing to our project, but unfortunately, not all contributions are suitable or aligned with our product roadmap. We hope you understand our decision, and please don't let it discourage you from contributing to open source projects in the future. We value all of our community members and their contributions, and we encourage you to keep exploring new projects and ways to get involved. For additional resources and information, please see the links below:
Thank you for your contributions to YOLO 🚀 and Vision AI ⭐ |
@xiang-wuu thanks for your contribution! Integrating YOLOv5 with Gradio and hosting it with Hugging Face Spaces is an exciting addition. We appreciate your effort in providing references for the PR. To ensure a smooth review process, could you provide before and after profiling/inference/training results, if applicable? This will help us quantify the improvement your PR provides and evaluate its impact more effectively. Kindly refer to our Contributing Guide for more details on the review process. We value your contributions to YOLOv5 and look forward to reviewing your PR. For further assistance, feel free to explore our Ultralytics Docs. Thank you! |
This PR is intended to integrate yolov5 infer module using gradio and publish them with hugging face spaces to host the gradio app for quick inference as a web demo, this PR essentially contains
app.py
file which is required to run using gradio container and for now the entire code-base of ultralytics/yolov5/blob/master with added gradioapp.py
is hosted at spaces/xiang-wuu/yolov5 so the same code base can be tracked with any gradio app hosted with hugging face spaces repo, however the demo app is hosted at spaces/xiang-wuu/yolov5 any subsequent yolov5 version specific changes can be hosted using this single gradio app by raising the PR at spaces/xiang-wuu/yolov5/pull. As tracking the same code-base at hugging face spaces repo can be effectively done by maintaining separate git remotes , but there is only one issue, as hugging face spaces requiresyaml
configuration header at top ofREADME
in root project directory which is unfortunately rendered by the markdown parser used by GitHub , hence for this PR i commented out the header part inREADME
file for remote pointing to ultralytics/yolov5 but the same is needed for the remote pointing to git repo hosted with hugging face spaces as it can be seen here.References:
🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
Integration of Gradio interface for the YOLOv5 object detection model.
📊 Key Changes
app.py
file added for easy Gradio web interface launching.requirements.txt
.🎯 Purpose & Impact