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

chore: update OpenCV version in openvino-tensor-converter #88

Merged
merged 1 commit into from
Feb 6, 2024

Conversation

abrown
Copy link
Contributor

@abrown abrown commented Feb 5, 2024

The openvino-tensor-converter crate is a helpful extension to this project but not crucial, thus it does not receive as much attention as other parts. It is useful for generating tensor files of various shapes from images (e.g., for test fixtures), so I haven't felt the need to remove it yet. Its dependencies had fallen far out of date and included a very old version of shlex that has security vulnerabilities. By updating to the latest opencv crate version, this dependency is also updated. This change also updates the converter code to fit the latest opencv API shape and cargo-clippy guidance.

The `openvino-tensor-converter` crate is a helpful extension to this
project but not crucial, thus it does not receive as much attention as
other parts. It is useful for generating tensor files of various shapes
from images (e.g., for test fixtures), so I haven't felt the need to
remove it yet. Its dependencies had fallen far out of date and included
a very old version of `shlex` that has security vulnerabilities. By
updating to the latest `opencv` crate version, this dependency is also
updated. This change also updates the converter code to fit the latest
`opencv` API shape and `cargo-clippy` guidance.
@abrown abrown merged commit 7599a31 into intel:main Feb 6, 2024
15 checks passed
@abrown abrown deleted the update-opencv branch February 6, 2024 15:48
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