search

Xilinx / inference-server

https://xilinx.github.io/inference-server/
Apache License 2.0
43 stars 13 forks source link

Bug fix in migraphx filename mangling #202

Closed bpickrel closed 1 year ago

bpickrel commented 1 year ago

Summary of Changes

One-line fix for a bug in filename handling. An incorrect extension was added to migraphx model file names that contain more than one period.

Draft PR because I haven't run the test scripts.

gbuildx commented 1 year ago

Can one of the admins verify this patch?

varunsh-xilinx commented 1 year ago

test this please

varunsh-xilinx commented 1 year ago

ok to test

varunsh-xilinx commented 1 year ago

retest this please

abalasa-amd commented 1 year ago

ok to test

gbuildx commented 1 year ago

Build failed!

varunsh-xilinx commented 1 year ago

Check failed for VAI for a used port. The MIGraphX test passed.