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

Refactor to remove mask-to-polygons dependency #1275

Closed
Tracked by #1460
AdeelH opened this issue Sep 16, 2021 · 0 comments · Fixed by #1470
Closed
Tracked by #1460

Refactor to remove mask-to-polygons dependency #1275

AdeelH opened this issue Sep 16, 2021 · 0 comments · Fixed by #1470

Comments

@AdeelH
Copy link
Collaborator

AdeelH commented Sep 16, 2021

🚀 Feature

RV uses https://github.com/azavea/mask-to-polygons for vectorization, but the library is no longer being maintained. We should move the vectorization code into RV itself and remove this dependency.

Motivation

The library is no longer being maintained.

Pitch

We should move the vectorization code into RV itself and remove this dependency.

Alternatives

Alternatively, we can continue to use that library in its current state. But there is at least one possible improvement that can be made to that library: azavea/mask-to-polygons#17.

Additional context

N/A

@AdeelH AdeelH mentioned this issue Aug 23, 2022
23 tasks
@AdeelH AdeelH added dependencies Pull requests that update a dependency file and removed dependencies Pull requests that update a dependency file labels Sep 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant