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

Gardio integration with hugging face spaces #8784

Closed
wants to merge 17 commits into from

Conversation

xiang-wuu
Copy link

@xiang-wuu xiang-wuu commented Jul 29, 2022

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 gradio app.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 requires yaml configuration header at top of README 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 in README 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

  • README update: Added Gradio SDK metadata and corresponding Gradio badge.
  • New Gradio app: app.py file added for easy Gradio web interface launching.
  • Requirements update: Added specific Gradio version requirement to requirements.txt.

🎯 Purpose & Impact

  • Enhanced accessibility: Users can now quickly test YOLOv5 with a web interface provided by Gradio, facilitating a hands-on experience without deep technical knowledge.
  • Extended compatibility: The pinned protobuf version ensures compatibility with existing systems.
  • Potential for growth: The Gradio integration can lead to increased exposure and use of YOLOv5 by making it more approachable for beginners and non-developers.🚀

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 @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

@xiang-wuu
Copy link
Author

pre-commit.ci autofix

@xiang-wuu
Copy link
Author

@glenn-jocher can you review this PR.

@github-actions
Copy link
Contributor

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 ⭐.

@github-actions github-actions bot added the Stale Stale and schedule for closing soon label Mar 22, 2023
@github-actions github-actions bot removed the Stale Stale and schedule for closing soon label Apr 10, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Oct 3, 2023

👋 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 ⭐

@github-actions github-actions bot added the Stale Stale and schedule for closing soon label Oct 3, 2023
@github-actions github-actions bot closed this Nov 3, 2023
@glenn-jocher
Copy link
Member

@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!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Stale Stale and schedule for closing soon
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants