AMDMIGraphX: Resnext50 failing to run on MIGraphX Driver
While trying https://zenodo.org/record/6617879/files/resnext50_32x4d_fpn.onnx with…
migraphx-driver perf ./resnext50_32x4d_fpn.onnx
I hit an error indicating an operator is not supported by migraphx…
what(): /workspace/AMDMIGraphX/src/onnx/onnx_parser.cpp:318: parse_graph: Unknown operator: Mod
Investigate and add
About this issue
- Original URL
- State: closed
- Created 2 years ago
- Comments: 38
Commits related to this issue
- Add in changes for onnx Mod operator Initial operator for mod implimentation and test cases for integer and floating based types. Need to use fmod from stdlib for floating point types. half_float::h... — committed to ROCm/AMDMIGraphX by TedThemistokleous 2 years ago
- Initial Changes for gpu and cpu implimentation for mod and fmod Initial changes for gpu/cpu implementation to get recognized. Still needs some changes for tests and if HIP supports fmod and remainde... — committed to ROCm/AMDMIGraphX by TedThemistokleous 2 years ago
- Initial Changes for gpu and cpu implimentation for mod and fmod Initial changes for gpu/cpu implementation to get recognized. Still needs some changes for tests and if HIP supports fmod and remainde... — committed to ROCm/AMDMIGraphX by TedThemistokleous 2 years ago
- Initial Changes for gpu and cpu implimentation for mod and fmod Initial changes for gpu/cpu implementation to get recognized. Still needs some changes for tests and if HIP supports fmod and remainde... — committed to ROCm/AMDMIGraphX by TedThemistokleous 2 years ago
- Add onnx mod operator (#1302) * Add in changes for onnx Mod operator Initial operator for mod implementation and test cases for integer and floating based types. Need to use fmod from stdlib fo... — committed to ROCm/AMDMIGraphX by TedThemistokleous 2 years ago
- Initial Changes for gpu and cpu implimentation for mod and fmod Initial changes for gpu/cpu implementation to get recognized. Still needs some changes for tests and if HIP supports fmod and remainde... — committed to ROCm/AMDMIGraphX by TedThemistokleous 2 years ago
- Add tupleVisitor for from_gpu (#1465) Need this for when we debug and use MIGRAPHX_TRACE_EVAL() to show tuples Without this we break when reading our buffer due to the use of visit() This came up a... — committed to ROCm/AMDMIGraphX by TedThemistokleous 2 years ago
- Adding testcase for large max boxes and sanity nms test Realize we didn't actually have a compute test for nonmaxsuppression to just run this during our test runs. Add it as a part of comparrison wit... — committed to ROCm/AMDMIGraphX by TedThemistokleous a year ago
We’re finally getting runs on resnext50 thanks to @pfultz2 and hinding and fixing an issue with concat vectorization #1653 In the meantime i was trying to make our NMS multithreaded for a single huge batch thinking that was still the issue as we worked in parallel.
Adding both of these fixes ontop of each-other gives us the following result
We still have an issue with accuracy when using the accuracy checker and we’re currently seeing an HSA_FAULT when running with MIGRAPHX_GPU_DEBUG =1
which is something I’m looking into between reviews