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.

Sign up for GitHub

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

Jump to bottom

Add explicit opencv dependency #1575

Merged
merged 3 commits into from
May 25, 2023
Merged

Add explicit opencv dependency #1575

merged 3 commits into from
May 25, 2023

Conversation

KSGulin
Copy link
Contributor

@KSGulin KSGulin commented May 25, 2023

The current install spaseml[torchvision] command does not include opencv and will result in an error on runtime. This PR adds the requirement with the same version constraints as DeepSparse

@KSGulin KSGulin requested a review from a team May 25, 2023 15:33
@KSGulin KSGulin self-assigned this May 25, 2023
@KSGulin KSGulin requested review from mgoin, eldarkurtic and anmarques and removed request for a team May 25, 2023 15:33
@KSGulin KSGulin merged commit 494cd72 into main May 25, 2023
@KSGulin KSGulin deleted the add_opencv_dep branch May 25, 2023 17:34
KSGulin added a commit that referenced this pull request May 26, 2023
@KSGulin
Add explixit open-cv dep (#1575)
3dd3af9
markurtz pushed a commit that referenced this pull request May 26, 2023
@KSGulin
[Cherry-Pick] Doc fixes (#1583)
8d83570 
* YOLOv5 doc fixes (#1574)

* IC doc fixes (#1577)

* Add explixit open-cv dep (#1575)

* Transformers doc fixes (#1578)

* Fix yolov8 export alias (#1579)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rahul-tuli rahul-tuli rahul-tuli approved these changes

@dbogunowicz dbogunowicz dbogunowicz approved these changes

@mgoin mgoin Awaiting requested review from mgoin mgoin was automatically assigned from neuralmagic/machine-learning

@eldarkurtic eldarkurtic Awaiting requested review from eldarkurtic eldarkurtic was automatically assigned from neuralmagic/machine-learning

@anmarques anmarques Awaiting requested review from anmarques anmarques was automatically assigned from neuralmagic/machine-learning

Assignees

@KSGulin KSGulin

Labels
mle-team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@KSGulin @rahul-tuli @dbogunowicz