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

SegmentationModel - get proper output in postprocess() #143

Merged
merged 1 commit into from
Dec 3, 2023

Conversation

mgumowsk
Copy link
Collaborator

What does this PR do?

Using SegmentationModel with OpenVINO Model Server adapter caused ov::merge exception.
The reason is the order of the outputs.
This PR makes sure, that feature_vector is not used in postprocess() function.

@sovrasov
Copy link
Collaborator

sovrasov commented Dec 1, 2023

@mgumowsk apparently we have some CI issues, I'll make a dummy PR to check

@sovrasov
Copy link
Collaborator

sovrasov commented Dec 1, 2023

@mgumowsk we can merge this if it's urgent, since #144 fails on the same testcases.
I believe that happened because of a new version of ultralytics package and OV models converted with a new OV version

@mgumowsk
Copy link
Collaborator Author

mgumowsk commented Dec 1, 2023

@mgumowsk we can merge this if it's urgent, since #144 fails on the same testcases. I believe that happened because of a new version of ultralytics package and OV models converted with a new OV version

@sovrasov if you can then yes, please merge, without it segmentation model doesn't work in Geti.

@sovrasov sovrasov merged commit 5a9d0e0 into openvinotoolkit:master Dec 3, 2023
15 of 17 checks passed
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.

2 participants