-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[MMSIG]支持 DeblurGANv2 inference #1955
Conversation
demo 示例 python mmagic\demo\mmagic_inference_demo.py --model-name deblurganv2 --model-comfig ../configs/deblurganv2/deblurganv2_fpn_inception.py --model-ckpt 权重文件路径 --img 测试图片路径 --device cpu --result-out-dir ./out.png
We recommend using English or English & Chinese for pull requests so that we could have broader discussion. |
hi @xiaomile ,
|
fix CI test
fix CI test
fix CI test
update model-index
Fix CI Test
CI test fix and update readme.md
fix CI test and update readme.md
Fix CI test
yapf 修正
代码调整,参数名保持一致
doc string coverage
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #1955 +/- ##
==========================================
- Coverage 82.24% 80.87% -1.37%
==========================================
Files 416 423 +7
Lines 27955 28952 +997
Branches 4372 4461 +89
==========================================
+ Hits 22992 23416 +424
- Misses 4025 4579 +554
- Partials 938 957 +19
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
add some doc string
add some doc string
add unit test
fix unit test
hi @zengyh1900 ,
these steps were completed |
Co-authored-by: Yanhong Zeng <[email protected]>
Co-authored-by: Yanhong Zeng <[email protected]>
Co-authored-by: Yanhong Zeng <[email protected]>
Co-authored-by: Yanhong Zeng <[email protected]>
Co-authored-by: Yanhong Zeng <[email protected]>
Co-authored-by: Yanhong Zeng <[email protected]>
Co-authored-by: Yanhong Zeng <[email protected]>
Co-authored-by: Yanhong Zeng <[email protected]>
move the implementation of loss function to mmagic/models/losses add quick start to readme
fix unit test
re run unit test
hi @zengyh1900 ,requested changes were completed. |
Co-authored-by: Yanhong Zeng <[email protected]>
Co-authored-by: Yanhong Zeng <[email protected]>
Co-authored-by: Yanhong Zeng <[email protected]>
Co-authored-by: Yanhong Zeng <[email protected]>
Co-authored-by: Yanhong Zeng <[email protected]>
Co-authored-by: Yanhong Zeng <[email protected]>
Co-authored-by: Yanhong Zeng <[email protected]>
Co-authored-by: Yanhong Zeng <[email protected]>
fix some url and path add README_zh-CN add Deblurring task into mmagic/apis/inferencers/__init__.py
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.
@xiaomile It is not easy work, but you have achieved it! Thanks for your great contribution!
Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
Please describe the motivation of this PR and the goal you want to achieve through this PR.
Modification
Please briefly describe what modification is made in this PR.
BC-breaking (Optional)
Does the modification introduce changes that break the backward-compatibility of the downstream repositories?
If so, please describe how it breaks the compatibility and how the downstream projects should modify their code to keep compatibility with this PR.
Use cases (Optional)
If this PR introduces a new feature, it is better to list some use cases here, and update the documentation.
Checklist
Submitting this pull request means that,
Before PR:
After PR: