xmos / ai_tools

AI applications and tools
Other
25 stars 10 forks source link

Lower strided slice #875

Closed michaelpoluektov closed 9 months ago

michaelpoluektov commented 9 months ago

This should fix the Jenkins fail in #873 because there is potentially a bug in TFL strided slice reference op

michaelpoluektov commented 9 months ago

GH actions failed on linux for the same reason as the previous PR: because the regular (reference) strided slice is (potentially) broken on linux for whatever reason

strided_slice_4.tflite and strided_slice_5.tflite are both cases that are NOT be covered by the new pass (both have strides that aren't 1), and the fail happens after the passes are actually ran (on AllocateTensors)

These tests would also fail on the current version of develop: the reason CI fails now is because the tests were added and not because something extra got broken.

What do you want to do about it?