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

OpenVX 1.3 Support #372

Merged
merged 107 commits into from
Mar 15, 2021
Merged

OpenVX 1.3 Support #372

merged 107 commits into from
Mar 15, 2021

Conversation

kiritigowda
Copy link
Collaborator

@kiritigowda kiritigowda commented Oct 5, 2020

  • OpenVX 1.3 CPU Support
  • Minor Bug Fixes
  • Vision Conformance Profile

Use CMakeList Variable OPENCL_SUPPORT

cmake -D OPENCL_SUPPORT=OFF ../MIVisionX

kiritigowda and others added 30 commits December 17, 2019 14:17
* changes for object-array

* threshold functions & adding objectarray to ago_util

* remap functions

* advanced array functions

* vxCreateMatrixFromPatternAndOrigin

* setImagePixelValues
* createvirtualscalar

* vxCreateVirtualConvolution

* syntax fix

* vxCreateVirtiaulDistribution, scalar syntax fix

* vxCreateVirtiaulDistribution, scalar syntax fix

* vxCreateVirtiaulMatrix

* vxWeightedAverageNode vxuWeightedAverage

* vxNonLinearFilterNode, vxuNonLinearFilter

* vxLaplacianPyramidNode, vxuLaplacianPyramid

* vxLaplacianReconstructNode, vxuLaplacianReconstruct

* type fix
* createvirtualscalar

* vxCreateVirtualConvolution

* syntax fix

* vxCreateVirtiaulDistribution, scalar syntax fix

* vxCreateVirtiaulDistribution, scalar syntax fix

* vxCreateVirtiaulMatrix

* vxWeightedAverageNode vxuWeightedAverage

* vxNonLinearFilterNode, vxuNonLinearFilter

* vxLaplacianPyramidNode, vxuLaplacianPyramid

* vxLaplacianReconstructNode, vxuLaplacianReconstruct

* type fix

* fixes vxGetUserStructNameByEnm and EnumByName

* fixes vxCreateObjectArray and vxCreateVirtualObjectArray

* conformance test nodes

* bug ifx

* declaration added

* threshold functions

* fixes matrix functions

* fixes vxSetImagePixelValues

* fixes remap functions

* threshold typo fix

* changed all of objectarray to replicate delay

* threshold kernels

* weighted average invalid format fix

* fixes vxCreateVirtualScalar

* fixes vxCReateVirtualConvolution

* fixes vxCopyLUT & vxMapLUT

* fixing

* weightedaverage passed

* fix comment

* graph/context/refernce+ fixes

* target base passes all - vx_context

* waitGraph and verifyGraphBase

* undo verifyGraphBase

* nonlinearfilter passing

* latest chanegs

* Update CMakeLists.txt

* Update ago_platform.h

* Update vx_api.cpp

* moving #def from VX/include to vx_ext_amd.h

* moving #def from VX/include to vx_ext_amd.h

* adjusting spaces

Co-authored-by: Hansel Yang <hanselyang123@gmail.com>
* createvirtualscalar

* vxCreateVirtualConvolution

* syntax fix

* vxCreateVirtiaulDistribution, scalar syntax fix

* vxCreateVirtiaulDistribution, scalar syntax fix

* vxCreateVirtiaulMatrix

* vxWeightedAverageNode vxuWeightedAverage

* vxNonLinearFilterNode, vxuNonLinearFilter

* vxLaplacianPyramidNode, vxuLaplacianPyramid

* vxLaplacianReconstructNode, vxuLaplacianReconstruct

* type fix

* fixes vxGetUserStructNameByEnm and EnumByName

* fixes vxCreateObjectArray and vxCreateVirtualObjectArray

* conformance test nodes

* bug ifx

* declaration added

* threshold functions

* fixes matrix functions

* fixes vxSetImagePixelValues

* fixes remap functions

* threshold typo fix

* changed all of objectarray to replicate delay

* threshold kernels

* weighted average invalid format fix

* fixes vxCreateVirtualScalar

* fixes vxCReateVirtualConvolution

* fixes vxCopyLUT & vxMapLUT

* fixing

* weightedaverage passed

* fix comment

* graph/context/refernce+ fixes

* target base passes all - vx_context

* waitGraph and verifyGraphBase

* undo verifyGraphBase

* nonlinearfilter passing

* latest chanegs

* Update CMakeLists.txt

* Update ago_platform.h

* Update vx_api.cpp

* moving #def from VX/include to vx_ext_amd.h

* moving #def from VX/include to vx_ext_amd.h

* adjusting spaces

* moving things from openvx/include to ext_amd

* laplacian update

* disable opencl

* Update CMakeLists.txt

* Update ago_interface.cpp

* Update vx_api.cpp

* LaplacianReconstruct pass

* bug fix

* bracket fix

* remove debug comments

* Update ago_haf_cpu_generic_functions.cpp

Co-authored-by: Hansel Yang <hanselyang123@gmail.com>
* remove debug comments

* Update ago_haf_cpu_generic_functions.cpp

* changes to threshold

Co-authored-by: Hansel Yang <hanselyang123@gmail.com>
* LaplacianPyramid Pass

* CMake fix

* Revert change

Co-authored-by: Hansel Yang <hanselyang123@gmail.com>
* Revert change

* opencl changes to threshold

* threshold with C code

Co-authored-by: Hansel Yang <hanselyang123@gmail.com>
* fixes meanstddev



Co-authored-by: Hansel Yang <hanselyang123@gmail.com>
* usernode 20 out of 74 tests pass

* removing printf statements

* spacing fix

Co-authored-by: Hansel Yang <hanselyang123@gmail.com>
* spacing fix

* adding verification path to fix user node

* fixes all tests of the user node

Co-authored-by: Hansel Yang <hanselyang123@gmail.com>
* fixes all tests of userNode

* swapImageHandle - roi=false cases

Co-authored-by: Hansel Yang <hanselyang123@gmail.com>
* swapImageHandle - roi=false cases

* fixes replicateNode object array

Co-authored-by: Hansel Yang <hanselyang123@gmail.com>
* fixes LUT failures

Co-authored-by: Hansel Yang <hanselyang123@gmail.com>
* wait graph fixed

* deadlock debug

* deadlock fixed

* cmake fix

* comments added

Co-authored-by: LakshmiKumar23 <kumar.lakshmi1994@gmail.com>
@paveltc
Copy link
Contributor

paveltc commented Jan 12, 2021

@kiritigowda This PR passed the smoke test.

@@ -1362,17 +1507,23 @@ int agoGetDataFromDescription(AgoContext * acontext, AgoGraph * agraph, AgoData
data->ref.type = VX_TYPE_THRESHOLD;
const char *s = strstr(desc, ","); if (!s) return -1;
char thresh_type[64], data_type[64];
uint32_t input_format, output_format;
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@hansely & @LakshmiKumar23 - this section of the code is buggy, try runvx canny to test.

Copy link
Contributor

Choose a reason for hiding this comment

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

@kiritigowda Found the issue. Emailed the details

@rrawther
Copy link
Contributor

rrawther commented Feb 9, 2021

@kiritigowda : What is pending for merging this PR?

@kiritigowda
Copy link
Collaborator Author

@kiritigowda : What is pending for merging this PR?

@rrawther this is failing the below test, @paveltc needs to push RunVX changes that support threshold to merge this

Running OpenVX Node - canny_edge_detector-1080p-u8
runvx 1.0.0
OK: using AMD OpenVX 1.3.0
terminate called after throwing an instance of 'std::invalid_argument'
  what():  stoi
Aborted (core dumped)

@paveltc
Copy link
Contributor

paveltc commented Feb 9, 2021

@rrawther and @kiritigowda I'm still working on updating the threshold object in runvx. I'm hoping to have something checked in later today.

kiritigowda and others added 12 commits February 9, 2021 10:47
Co-authored-by: paveltc <pavel.tcherniaev@amd.com>
* CMake

* multiply fix

* code cleanup
* CMake

* canny fix

* code cleanup
* fixes GraphROI.Simple & vxMapRemapPatch.MapRandomRemap

* Graph.GraphState

* fixes Threshold.OnRandom/4/Graph/BINARY/U8/U8

* removing unwanted commits

* fixes Threshold.OnRandom/5/Graph/BINARY/S16/U8

* fixes Threshold.OnRandom/7/Graph/RANGE/S16/U8

* removing unnecessary changes
* channel combine fix

* merge fix
@kiritigowda kiritigowda merged commit 4aeb30f into ROCm:master Mar 15, 2021
@kiritigowda kiritigowda deleted the kg/openvx-1.3-port branch March 15, 2021 07:30
@kiritigowda kiritigowda restored the kg/openvx-1.3-port branch March 15, 2021 07:30
@rrawther
Copy link
Contributor

@kiritigowda I think we should fix some of that codecy warnings since most of them are for unused variables

@kiritigowda kiritigowda deleted the kg/openvx-1.3-port branch April 21, 2021 22:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Bug fixes to existing features New Feature New Features added to MIVisionX
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants