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

Add feature for image demo. #128

Merged
merged 12 commits into from
Oct 12, 2022
Merged

Add feature for image demo. #128

merged 12 commits into from
Oct 12, 2022

Conversation

JiayuXu0
Copy link
Contributor

@JiayuXu0 JiayuXu0 commented Oct 3, 2022

Motivation

User has issue in the demo.

Modification

Add the switch of whether to display the result. The default value is False.

Add the inference test of the images folder.

Add progress bar in image demo.

@JiayuXu0
Copy link
Contributor Author

JiayuXu0 commented Oct 3, 2022

The tests failed because flake8 version is conflict with importlib_metadata. It have been fixed in #129

@JiayuXu0 JiayuXu0 changed the title Add the switch of whether to display the result. Add feature for image inference demo. Oct 4, 2022
@JiayuXu0 JiayuXu0 changed the title Add feature for image inference demo. Add feature for image and video inference demo. Oct 6, 2022
demo/image_demo.py Outdated Show resolved Hide resolved
docs/en/get_started.md Outdated Show resolved Hide resolved
demo/image_demo.py Outdated Show resolved Hide resolved
demo/image_demo.py Outdated Show resolved Hide resolved
demo/image_demo.py Outdated Show resolved Hide resolved
demo/image_demo.py Outdated Show resolved Hide resolved
@hhaAndroid
Copy link
Collaborator

@JiayuXu0 The specific usage of the script that needs to be written in the pr message

mmyolo/registry.py Outdated Show resolved Hide resolved
demo/image_demo.py Outdated Show resolved Hide resolved
demo/image_demo.py Outdated Show resolved Hide resolved
demo/image_demo.py Outdated Show resolved Hide resolved
@hhaAndroid hhaAndroid mentioned this pull request Oct 9, 2022
32 tasks
@JiayuXu0 JiayuXu0 requested review from PeterH0323 and hhaAndroid and removed request for PeterH0323 and hhaAndroid October 9, 2022 14:04
demo/image_demo.py Outdated Show resolved Hide resolved
demo/image_demo.py Outdated Show resolved Hide resolved
demo/video_demo.py Outdated Show resolved Hide resolved
demo/video_demo.py Outdated Show resolved Hide resolved
demo/video_demo.py Outdated Show resolved Hide resolved
demo/image_demo.py Outdated Show resolved Hide resolved
mmyolo/registry.py Outdated Show resolved Hide resolved
mmyolo/registry.py Outdated Show resolved Hide resolved
demo/video_demo.py Outdated Show resolved Hide resolved
demo/video_demo.py Outdated Show resolved Hide resolved
demo/video_demo.py Outdated Show resolved Hide resolved
demo/video_demo.py Outdated Show resolved Hide resolved
demo/video_demo.py Outdated Show resolved Hide resolved
demo/video_demo.py Outdated Show resolved Hide resolved
demo/image_demo.py Show resolved Hide resolved
demo/image_demo.py Outdated Show resolved Hide resolved
demo/image_demo.py Outdated Show resolved Hide resolved
demo/image_demo.py Outdated Show resolved Hide resolved
demo/image_demo.py Outdated Show resolved Hide resolved
@hhaAndroid hhaAndroid changed the title Add feature for image and video inference demo. Add feature for image demo. Oct 10, 2022
@ztyf-lq ztyf-lq mentioned this pull request Oct 10, 2022
demo/image_demo.py Outdated Show resolved Hide resolved
demo/image_demo.py Outdated Show resolved Hide resolved
docs/zh_cn/get_started.md Outdated Show resolved Hide resolved
docs/zh_cn/get_started.md Outdated Show resolved Hide resolved
docs/en/get_started.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@PeterH0323 PeterH0323 left a comment

Choose a reason for hiding this comment

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

LGTM

@hhaAndroid hhaAndroid merged commit c4497db into open-mmlab:dev Oct 12, 2022
hhaAndroid pushed a commit to hhaAndroid/mmyolo that referenced this pull request Oct 13, 2022
* Add the switch of whether to display the result.

* Add the inference test of the images folder

* Add video demo.

* Add progress bar in image and video demo.

* Add dir inference support.

* Add url download support.

* Add output path prompt at the end.

* Modify the logic of saving results.

* Update mmengine limit.
hhaAndroid pushed a commit to hhaAndroid/mmyolo that referenced this pull request Oct 13, 2022
* Add the switch of whether to display the result.

* Add the inference test of the images folder

* Add video demo.

* Add progress bar in image and video demo.

* Add dir inference support.

* Add url download support.

* Add output path prompt at the end.

* Modify the logic of saving results.

* Update mmengine limit.
hhaAndroid pushed a commit that referenced this pull request Oct 13, 2022
* Add the switch of whether to display the result.

* Add the inference test of the images folder

* Add video demo.

* Add progress bar in image and video demo.

* Add dir inference support.

* Add url download support.

* Add output path prompt at the end.

* Modify the logic of saving results.

* Update mmengine limit.
hhaAndroid pushed a commit that referenced this pull request Oct 13, 2022
* Add the switch of whether to display the result.

* Add the inference test of the images folder

* Add video demo.

* Add progress bar in image and video demo.

* Add dir inference support.

* Add url download support.

* Add output path prompt at the end.

* Modify the logic of saving results.

* Update mmengine limit.
hhaAndroid pushed a commit that referenced this pull request Oct 13, 2022
* Add the switch of whether to display the result.

* Add the inference test of the images folder

* Add video demo.

* Add progress bar in image and video demo.

* Add dir inference support.

* Add url download support.

* Add output path prompt at the end.

* Modify the logic of saving results.

* Update mmengine limit.
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.

3 participants