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

graph.nnef Parse error #8

Closed
srohit0 opened this issue Dec 15, 2018 · 3 comments
Closed

graph.nnef Parse error #8

srohit0 opened this issue Dec 15, 2018 · 3 comments
Assignees
Labels
bug Something isn't working enhancement New feature or request

Comments

@srohit0
Copy link

srohit0 commented Dec 15, 2018

Using latest amdovx-modules repo to convert nnef to openvx gives me this error

OK: parsing /home/amd/models/nnef/object_detection//mobilenet_v1_1/graph.nnef ...
Parse error: [2:12] expected token 'graph', found ';'

NNEF model excerpt

 
 version 1.0;

 graph mobilenet_v1_1( input ) -> ( output )
 {

Script

MODEL=mobilenet_v1_1
IN_DIR=/home/amd/models/nnef/object_detection/
OUT_DIR=/home/amd/models/openvx/object_detection/

mkdir ${OUT_DIR}/$MODEL
nnef2openvx ${IN_DIR}/$MODEL ${OUT_DIR}/$MODEL

Output

OK: parsing /home/amd/models/nnef/object_detection//mobilenet_v1_1/graph.nnef ...
Parse error: [2:12] expected token 'graph', found ';'

NNEF specification

Section 3.2.4 of NNEF Specification Version 1.0, Revision 3, 2018-06-13 states that semicolon ; is part of version statement. Here is excerpt from the specification:

The version info is introduced by the version keyword, and is defined by a real-number numeric literal as major and minor versions separated by a dot:

<version> ::= "version" <numeric-literal> ";"

Fix is needed to support NNEF Specification Version 1.0, Revision 3, 2018-06-13

@kiritigowda kiritigowda transferred this issue from GPUOpen-ProfessionalCompute-Libraries/amdovx-modules Jan 5, 2019
@kiritigowda kiritigowda added enhancement New feature or request bug Something isn't working labels Jan 5, 2019
@hansely
Copy link
Contributor

hansely commented Feb 14, 2019

This issue will be resolved after the nnir_to_nnef.py fix pull request.

@kiritigowda
Copy link
Collaborator

@hansely is this issue fixed?

@kiritigowda
Copy link
Collaborator

refer to issue #54

hansely referenced this issue in hansely/MIVisionX May 7, 2019
* nnir_to_openvx print (ROCm#103)

* Ubuntu 18.04 warnings fix (ROCm#104)

* Support for ONNX V1.3 (ROCm#101)

ONNX 1.3 support for ResNet50

* perf_chart update
LakshmiKumar23 referenced this issue in LakshmiKumar23/MIVisionX Mar 31, 2020
hansely referenced this issue in hansely/MIVisionX Apr 10, 2020
kiritigowda added a commit that referenced this issue Mar 15, 2021
* OpenVX 1.3 Headers Added

* Temp Changes to build MIVisionX with OpenVX 1.3 Headers

* VX NN 1.3 Port

* Lk/openvx port 1.3 (#2)

* changes for object-array

* threshold functions & adding objectarray to ago_util

* remap functions

* advanced array functions

* vxCreateMatrixFromPatternAndOrigin

* setImagePixelValues

* OpenVX 1.3 Conformance build

* createvirtualscalar

* vxCreateVirtualConvolution

* syntax fix

* vxCreateVirtiaulDistribution, scalar syntax fix

* vxCreateVirtiaulDistribution, scalar syntax fix

* vxCreateVirtiaulMatrix

* vxWeightedAverageNode vxuWeightedAverage

* vxNonLinearFilterNode, vxuNonLinearFilter

* vxLaplacianPyramidNode, vxuLaplacianPyramid

* vxLaplacianReconstructNode, vxuLaplacianReconstruct

* type fix

* latest changes (#5)

* 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>

* Lk/port1.3 fix new (#6)

* 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>

* changes from Hansel and threshold changes (#7)

* remove debug comments

* Update ago_haf_cpu_generic_functions.cpp

* changes to threshold

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

* OpenVX 1.3 port - Laplacian Pyramid (#8)

* Laplacian pyramid fix (#9)

* LaplacianPyramid Pass

* CMake fix

* Revert change

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

* OpenVX Port - fix build failures (#10)

* Revert change

* opencl changes to threshold

* threshold with C code

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

* mean std deviation fix  (#11)

* fixes meanstddev

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

* user node 20/74 tests passes  (#12)

* usernode 20 out of 74 tests pass

* removing printf statements

* spacing fix

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

* USerNode: All tests pass (#13)

* spacing fix

* adding verification path to fix user node

* fixes all tests of the user node

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

* SwapImageHandle roi=false cases passes (#14)

* fixes all tests of userNode

* swapImageHandle - roi=false cases

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

* Fixes ReplicateNode-ObjectArray (#17)

* swapImageHandle - roi=false cases

* fixes replicateNode object array

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

* fixes LUT type=S16 failures (#18)


* fixes LUT failures

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

* H/openvx 1.3 port new (#19)

* wait graph fixed

* deadlock debug

* deadlock fixed

* cmake fix

* comments added

Co-authored-by: LakshmiKumar23 <kumar.lakshmi1994@gmail.com>

* OpenVX 1.3 Fix

* weighted average validation fix

* graphstate fixed

* check fix

* bug fix

* SmokeTestBase.vxReleaseReferenceBase fix (#21)

* SmokeTestBase.vxReleaseReferenceBase fix

* SmokeTestBase.vxSetReferenceName fix

* Graph State - Fix & code cleanup (#22)

* Graph Tests - Fix (#23)

* code cleanup

* user node fixes

* graph state fixes

* smoketestbase - vxRetainReferenceBase fix (#24)

* smoketestbase-vxRetainReferenceBase fix

* histogram - Fix CTS Errors(#25)

* debugging histogram

* histogram fix

* code cleanup

* Resource Release Fix

* OpenVX 1.3 - half scale gaussian (#27)

* halfscalegaussian fix

* code cleanup

* OpenVX 1.3 - smokeTest.vxRetainReference & smokeTestBase.vxSetReferenceName fix (#26)

* vxRetainRef line 371  fix

* vxSetReferenceName fix

* pyramid fixes

* smoke test fixes for dangling references

* resolves all dangling reference issues

* vxUnloadKernels Fix (#28)

* CTS - graph delay with pyramid fix (#29)

* Canny - CTS bug fix (#30)

* divide by 4 when grad size 7

* canny fix

* merge

* canny fix

* code cleanup

* Run OpenVX 1.3 CPU Conformance

* Travis Fix - OpenVX 1.3

* Travis Cleanup

* OpenVX 1.3 - Harris corner CTS Fix (#31)

* CMake

* harris fix

* code cleanup

* swapImageHandle  (#32)

* changing buffers to reflect right values

* getting correct pointers for ROI

* buffer values for multiple plane images

* fixes 12/39 swapImageHandle cases

* fixes all swapImageHandle errors

* fixes color convert  errors (#33)

* fixes conversion from RGBX to NV12&IYUV

* fixes conversion from UYVY to NV12&IYUV

* fixes conversion from YUYV to NV12&IYUV

* fixes converion from NV12,NV21&IYUV to RGBX

* fixes conversion from NV12,NV21&IYUV to RGB

* warp affine - fix (#34)

* warp affine fix

* code cleanup

* Travis - Trace Error

* Travis - Check VGA

* Travis Updates

* Fix - variable scope

* CXX Flags & OpenVX Version Update

* changed Threshold to support new OpenVX 1.3 format (#38)

Co-authored-by: paveltc <pavel.tcherniaev@amd.com>

* Threshold - Update to 1.3

* Jenkins - Check Build & Artifacts

* Tests - Fix platform name

* GPU Fix - multiply gpu (#39)

* CMake

* multiply fix

* code cleanup

* GPU Flow - Canny Fix (#36)

* CMake

* canny fix

* code cleanup

* GPU Flow - Bug Fixes (#35)

* 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

* GPU Flow - fixes LUT for data type s16 (#40)

* GPU Flow - channel combine (#41)

* channel combine fix

* merge fix

Co-authored-by: LakshmiKumar23 <kumar.lakshmi1994@gmail.com>
Co-authored-by: Hansel Yang <hanselyang123@gmail.com>
Co-authored-by: Kiriti Gowda <kiriti@Kiritis-MacBook-Pro.local>
Co-authored-by: LakshmiKumar23 <lakshmi.kumar@amd.com>
Co-authored-by: Hansel Yang <hansyang@amd.com>
Co-authored-by: Kiriti Nagesh Gowda <kiritigowda@Kiritis-MacBook-Pro.local>
Co-authored-by: Pavel Tcherniaev <ptcherni@amd.com>
Co-authored-by: paveltc <pavel.tcherniaev@amd.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants