Skip to content

Added PaddlePaddle classification model conversion case#19976

Merged
alalek merged 4 commits intoopencv:masterfrom
jiangjiajun:master
May 13, 2021
Merged

Added PaddlePaddle classification model conversion case#19976
alalek merged 4 commits intoopencv:masterfrom
jiangjiajun:master

Conversation

@jiangjiajun
Copy link
Copy Markdown
Contributor

@jiangjiajun jiangjiajun commented Apr 26, 2021 •
edited by alalek
Loading

Provided dnn_model_runner module part for the classification model of PaddlePaddle and inference with OpenCV API

Pull Request Readiness Checklist

See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request

  • I agree to contribute to the project under Apache 2 License.
  • To the best of my knowledge, the proposed patch is not based on a code under GPL or other license that is incompatible with OpenCV
  • The PR is proposed to proper branch
  • There is reference to original bug report and related work
  • There is accuracy test, performance test and test data in opencv_extra repository, if applicable
    Patch to opencv_extra has the same branch name.
  • The feature is well documented and sample code can be built with the project CMake

force_builders=linux,docs

vpisarev
vpisarev reviewed Apr 30, 2021
View reviewed changes
samples/dnn/dnn_model_runner/dnn_conversion/paddlepaddle/paddle_resnet50.py Outdated
import paddle
import paddlehub as hub
import paddlehub.vision.transforms as T
import cv2
Copy link
Copy Markdown
Contributor

@vpisarev vpisarev Apr 30, 2021

Choose a reason for hiding this comment

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

please, replace it with import cv2 as cv and then replace cv2. below with cv.

Copy link
Copy Markdown
Contributor Author

@jiangjiajun jiangjiajun May 6, 2021

Choose a reason for hiding this comment

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

Done

@vpisarev vpisarev self-assigned this May 12, 2021
@vpisarev
Copy link
Copy Markdown
Contributor

vpisarev commented May 12, 2021

👍, @alalek, we can merge it, I think

@vpisarev vpisarev self-requested a review May 12, 2021 10:07
@alalek
Copy link
Copy Markdown
Member

alalek commented May 12, 2021

Need to fix whitespace issues: http://pullrequest.opencv.org/buildbot/builders/precommit_docs/builds/30369

@jiangjiajun
Copy link
Copy Markdown
Contributor Author

jiangjiajun commented May 13, 2021

Need to fix whitespace issues: http://pullrequest.opencv.org/buildbot/builders/precommit_docs/builds/30369

Problem solved.

@alalek alalek merged commit fea45c6 into opencv:master May 13, 2021
@alalek alalek mentioned this pull request Jun 4, 2021
Merged
a-sajjad72 pushed a commit to a-sajjad72/opencv that referenced this pull request Mar 30, 2023
@jiangjiajun
Merge pull request opencv#19976 from jiangjiajun:master
6be9982 
* Added PaddlePaddle classification model conversion case

* Modify cv2 import as cv

* Modify documents in dnn_conversion/paddlepaddle

* Modify documents in dnn_conversion/paddlepaddle
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vpisarev vpisarev vpisarev approved these changes

Assignees

@vpisarev vpisarev

Labels

category: dnn category: samples feature

Projects

None yet

Milestone

4.5.3

Development

Successfully merging this pull request may close these issues.

3 participants

@jiangjiajun @vpisarev @alalek