Skip to content

G-API: Extend GKernelPackage and serialization API#19617

Merged
alalek merged 4 commits intoopencv:masterfrom
smirnov-alexey:as/extend_kernel_package_api
Mar 10, 2021
Merged

G-API: Extend GKernelPackage and serialization API#19617
alalek merged 4 commits intoopencv:masterfrom
smirnov-alexey:as/extend_kernel_package_api

Conversation

@smirnov-alexey
Copy link
Copy Markdown
Contributor

@smirnov-alexey smirnov-alexey commented Feb 24, 2021 •
edited
Loading

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
Xforce_builders_only=linux,docs
force_builders=Custom,Custom Win,Custom Mac
build_gapi_standalone:Linux x64=ade-0.1.1f
build_gapi_standalone:Win64=ade-0.1.1f
build_gapi_standalone:Mac=ade-0.1.1f
build_gapi_standalone:Linux x64 Debug=ade-0.1.1f

build_image:Custom=centos:7
buildworker:Custom=linux-1
build_gapi_standalone:Custom=ade-0.1.1f

Xbuild_image:Custom=ubuntu-openvino-2021.1.0:20.04
build_image:Custom Win=openvino-2021.1.0
build_image:Custom Mac=openvino-2021.1.0

Xtest_modules:Custom=gapi
test_modules:Custom Win=gapi
test_modules:Custom Mac=gapi

Xbuildworker:Custom=linux-1
# disabled due high memory usage: test_opencl:Custom=ON
Xtest_opencl:Custom=OFF
Xtest_bigdata:Custom=1
Xtest_filter:Custom=*

@smirnov-alexey
Copy link
Copy Markdown
Contributor Author

smirnov-alexey commented Feb 24, 2021

@rgarnov @dmatveev fyi

smirnov-alexey
smirnov-alexey commented Feb 24, 2021
View reviewed changes
@smirnov-alexey smirnov-alexey changed the title G-API: Extend GKernelPackage API G-API: Extend GKernelPackage and serialization API Feb 26, 2021
smirnov-alexey
smirnov-alexey commented Feb 26, 2021
View reviewed changes
@smirnov-alexey
Copy link
Copy Markdown
Contributor Author

smirnov-alexey commented Mar 4, 2021

@rgarnov @dmatveev @AsyaPronina please, take a look

rgarnov
rgarnov reviewed Mar 9, 2021
View reviewed changes
Copy link
Copy Markdown
Contributor

@rgarnov rgarnov left a comment

Choose a reason for hiding this comment

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

Looks good, just a minor comments

AsyaPronina
AsyaPronina approved these changes Mar 9, 2021
View reviewed changes
Copy link
Copy Markdown
Contributor

@AsyaPronina AsyaPronina left a comment

Choose a reason for hiding this comment

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

If this will not be done automatically, please update copyrights in all participated files

@rgarnov rgarnov self-assigned this Mar 10, 2021
@smirnov-alexey
Copy link
Copy Markdown
Contributor Author

smirnov-alexey commented Mar 10, 2021

@alalek can we merge this? I seems like Ubuntu 20.04 build is not related to this PR

@alalek alalek merged commit ddd2447 into opencv:master Mar 10, 2021
AsyaPronina pushed a commit to AsyaPronina/opencv that referenced this pull request Mar 25, 2021
@smirnov-alexey @mshabunin
Merge pull request opencv#19617 from smirnov-alexey:as/extend_kernel_…
c102ac8 
…package_api

G-API: Extend GKernelPackage and serialization API

* Extend GKernelPackage API

* Adding tests

* Extend serialization API

* Address review comments
@alalek alalek mentioned this pull request Apr 9, 2021
Merged
a-sajjad72 pushed a commit to a-sajjad72/opencv that referenced this pull request Mar 30, 2023
@smirnov-alexey
Merge pull request opencv#19617 from smirnov-alexey:as/extend_kernel_…
fa04063 
…package_api

G-API: Extend GKernelPackage and serialization API

* Extend GKernelPackage API

* Adding tests

* Extend serialization API

* Address review comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AsyaPronina AsyaPronina AsyaPronina approved these changes

@rgarnov rgarnov rgarnov approved these changes

Assignees

@rgarnov rgarnov

Labels

category: g-api / gapi

Projects

None yet

Milestone

4.5.2

Development

Successfully merging this pull request may close these issues.

5 participants

@smirnov-alexey @AsyaPronina @rgarnov @asmorkalov @alalek