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

Remove OpenCV dependency #142

Closed
mpelchat04 opened this issue Jun 26, 2020 · 1 comment
Closed

Remove OpenCV dependency #142

mpelchat04 opened this issue Jun 26, 2020 · 1 comment
Labels
P3 Low priority
Milestone

Comments

@mpelchat04
Copy link
Collaborator

At installation time, OpenCV has a lot of conflictual dependencies with rasterio. fiona and scikit.

The functions that requires opencv in our code could be replaced with similar functions from scikit.

Plus, it's only used with distance map

@mpelchat04 mpelchat04 added the P3 Low priority label Aug 12, 2020
@valhassan valhassan added this to the 1.3.0 milestone Aug 13, 2020
@CharlesAuthier
Copy link
Collaborator

I think with the switch to hydra PR we can close this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P3 Low priority
Projects
None yet
Development

No branches or pull requests

3 participants