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

Fix OpenCV compatibility problems #1323

Merged
merged 3 commits into from
Jan 14, 2022
Merged

Fix OpenCV compatibility problems #1323

merged 3 commits into from
Jan 14, 2022

Conversation

AdeelH
Copy link
Collaborator

@AdeelH AdeelH commented Jan 13, 2022

Overview

This PR fixes the below error

  File "/opt/conda/lib/python3.7/site-packages/cv2/__init__.py", line 9, in <module>
    from .cv2 import _registerMatType
ImportError: cannot import name '_registerMatType' from 'cv2.cv2' (/opt/conda/lib/python3.7/site-packages/cv2/cv2.cpython-37m-x86_64-linux-gnu.so)

that started cropping up in recent builds during unit testing, even though no OpenCV-related changes have been made recently.

See other discussions of this error below:

Checklist

  • Added needs-backport label if PR is bug fix that applies to previous minor release
  • Ran scripts/format_code and committed any changes
  • Documentation updated if needed
  • PR has a name that won't get you publicly shamed for vagueness

Testing Instructions

  • build Docker image
  • Run unit tests
  • If unit tests succeed, the issue has been resolved.

@codecov
Copy link

codecov bot commented Jan 14, 2022

Codecov Report

Merging #1323 (e3c45ec) into master (fc181a6) will not change coverage.
The diff coverage is n/a.

❗ Current head e3c45ec differs from pull request most recent head b03fc51. Consider uploading reports for the commit b03fc51 to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1323   +/-   ##
=======================================
  Coverage   59.20%   59.20%           
=======================================
  Files         171      171           
  Lines        7935     7935           
=======================================
  Hits         4698     4698           
  Misses       3237     3237           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fc181a6...b03fc51. Read the comment docs.

@AdeelH
Copy link
Collaborator Author

AdeelH commented Jan 14, 2022

(Fixed a typo in one of the commit messages)

@AdeelH AdeelH merged commit 5a0eb7e into azavea:master Jan 14, 2022
@AdeelH AdeelH deleted the opencv branch January 14, 2022 07:36
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