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

Backwards compatibility between 0.19.x and 0.20.x is broken #1125

Closed
cospectrum opened this issue Jul 14, 2023 · 4 comments · Fixed by #1134
Closed

Backwards compatibility between 0.19.x and 0.20.x is broken #1125

cospectrum opened this issue Jul 14, 2023 · 4 comments · Fixed by #1134

Comments

@cospectrum
Copy link

Hi! Just noticed that yolov8 is no longer valid in tract-onnx 0.20.

Error: running pass PushSliceUp

Caused by:
    0: Calling slice on #410 "/model.22/Concat_4" Concat
    1: in output_facts invocation for /model.22/Concat_4.split-over-1.64..65.slice-1.64..65
    2: Condition failed: `end <= len` (65 vs 1)
@kali
Copy link
Collaborator

kali commented Jul 14, 2023

Thanks, that was of course not intentional. Can you link the onnx file you're using and the input fact ?

@cospectrum
Copy link
Author

Thanks, that was of course not intentional. Can you link the onnx file you're using and the input fact ?

yolov8n.onnx.zip
For example, the smallest detection model from ultralytics.
The input fact is (1, 3, 640, 640, F32).

Error: running pass PushSliceUp

Caused by:
    0: Calling slice on #410 "/model.22/Concat_3" Concat
    1: in output_facts invocation for /model.22/Concat_3.split-over-1.64..144.slice-1.64..144
    2: Condition failed: `end <= len` (144 vs 80)

@kali
Copy link
Collaborator

kali commented Aug 1, 2023

Hey, sorry for the long delay. Could you please check if #1134 fixes your issue ?

@kali kali closed this as completed in #1134 Aug 1, 2023
@cospectrum
Copy link
Author

Yes, I have no errors

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants