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

Bugfix/osrt cpp #92

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

erstrom
Copy link

@erstrom erstrom commented Oct 3, 2024

Various bugfixes found during tests with the ONNX CPP example application

…tenation

Add missing slash separator when creating artifacts path

Signed-off-by: Erik Stromdahl <[email protected]>
fix bug with index used when iterating the output tensor from output
segmentation

Signed-off-by: Erik Stromdahl <[email protected]>
Remove redundant tensor memory allocation (and memory leak). The memory
for the output will be allocated later in the output creation loop.

Signed-off-by: Erik Stromdahl <[email protected]>
Instead of reading width and height from indices 2 and 3, we read these
values from the last and second last indices of the shape vector. This
turns out to be indices 2 and 3 for most models.

Signed-off-by: Erik Stromdahl <[email protected]>
Print the values that are compared if there is a mismatch

Signed-off-by: Erik Stromdahl <[email protected]>
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 this pull request may close these issues.

1 participant