Skip to content

Remove legacy nGraph logic#24072

Merged
asmorkalov merged 2 commits intoopencv:4.xfrom
dkurt:openvino_cpu_tests
Aug 2, 2023
Merged

Remove legacy nGraph logic#24072
asmorkalov merged 2 commits intoopencv:4.xfrom
dkurt:openvino_cpu_tests

Conversation

@dkurt
Copy link
Copy Markdown
Member

@dkurt dkurt commented Jul 28, 2023 •
edited
Loading

Pull Request Readiness Checklist

TODO:

  • Test with OpenVINO 2021.4 (tested locally)

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 another license that is incompatible with OpenCV
  • The PR is proposed to the proper branch
  • There is a reference to the 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=Custom

Xbuild_image:Custom=ubuntu-openvino-2021.4.2:20.04
build_image:Custom=ubuntu-openvino-2022.1.0:20.04

test_modules:Custom=dnn,python2,python3,java,gapi,video

buildworker:Custom=linux-1
# disabled due high memory usage: test_opencl:Custom=ON
test_opencl:Custom=ON
test_bigdata:Custom=1
test_filter:Custom=*

@dkurt dkurt mentioned this pull request Jul 28, 2023
Merged
@opencv-alalek opencv-alalek added optimization cleanup Code cleanup (e.g, drop legacy C-API, legacy unmaintained code) labels Jul 29, 2023
@dkurt dkurt marked this pull request as ready for review July 30, 2023 16:31
@opencv-alalek opencv-alalek added this to the 4.9.0 milestone Aug 2, 2023
@asmorkalov asmorkalov requested review from asmorkalov and removed request for mshabunin and opencv-alalek August 2, 2023 11:18
asmorkalov
asmorkalov approved these changes Aug 2, 2023
View reviewed changes
Copy link
Copy Markdown
Contributor

@asmorkalov asmorkalov left a comment

Choose a reason for hiding this comment

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

👍

@asmorkalov asmorkalov merged commit 0245c0c into opencv:4.x Aug 2, 2023
@asmorkalov asmorkalov self-assigned this Aug 2, 2023
@dkurt dkurt deleted the openvino_cpu_tests branch August 2, 2023 11:39
@asmorkalov asmorkalov mentioned this pull request Aug 7, 2023
Merged
thewoz pushed a commit to thewoz/opencv that referenced this pull request Jan 4, 2024
@dkurt @thewoz
Merge pull request opencv#24072 from dkurt:openvino_cpu_tests
a0f473e 
Remove legacy nGraph logic opencv#24072

### Pull Request Readiness Checklist

TODO:
- [x] Test with OpenVINO 2021.4 (tested locally)

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

- [x] I agree to contribute to the project under Apache 2 License.
- [x] To the best of my knowledge, the proposed patch is not based on a code under GPL or another license that is incompatible with OpenCV
- [x] The PR is proposed to the proper branch
- [ ] There is a reference to the original bug report and related work
- [x] There is accuracy test, performance test and test data in opencv_extra repository, if applicable
      Patch to opencv_extra has the same branch name.
- [x] The feature is well documented and sample code can be built with the project CMake
thewoz pushed a commit to thewoz/opencv that referenced this pull request May 29, 2024
@dkurt @thewoz
Merge pull request opencv#24072 from dkurt:openvino_cpu_tests
ac87d7f 
Remove legacy nGraph logic opencv#24072

### Pull Request Readiness Checklist

TODO:
- [x] Test with OpenVINO 2021.4 (tested locally)

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

- [x] I agree to contribute to the project under Apache 2 License.
- [x] To the best of my knowledge, the proposed patch is not based on a code under GPL or another license that is incompatible with OpenCV
- [x] The PR is proposed to the proper branch
- [ ] There is a reference to the original bug report and related work
- [x] There is accuracy test, performance test and test data in opencv_extra repository, if applicable
      Patch to opencv_extra has the same branch name.
- [x] The feature is well documented and sample code can be built with the project CMake
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@asmorkalov asmorkalov asmorkalov approved these changes

Assignees

@asmorkalov asmorkalov

Labels

category: dnn cleanup Code cleanup (e.g, drop legacy C-API, legacy unmaintained code) optimization

Projects

None yet

Milestone

4.9.0

Development

Successfully merging this pull request may close these issues.

3 participants

@dkurt @asmorkalov @opencv-alalek