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

Union polygons #1

Closed
adnanefouam opened this issue Sep 28, 2020 · 6 comments
Closed

Union polygons #1

adnanefouam opened this issue Sep 28, 2020 · 6 comments
Labels
enhancement New feature or request

Comments

@adnanefouam
Copy link

hello, thanks for bringing turf to dart !
is there will be Union function that Takes two or more polygons and returns a combined polygon ?

@lukas-h
Copy link
Member

lukas-h commented Sep 29, 2020

This library is work in progress. Feel free to make a pull request, since it is not yet implemented.

@mohammedX6
Copy link

I have ported a libary that capable of doing all polygon Boolean operations on polygons (union, intersection, difference, xor) ( More info here

@lukas-h
Copy link
Member

lukas-h commented Apr 16, 2022

Hi @mohammedX6, this is really great! 😄
Do you want to create a pull request to make this part of turf_dart?

@mohammedX6
Copy link

I don't think i have time to make pull requests but you can you use the libary dierftly without need of dart turf

@lukas-h lukas-h added the enhancement New feature or request label Jul 7, 2022
@lukas-h
Copy link
Member

lukas-h commented Jan 30, 2024

closed because this is mostly implemented with the boolean* functions

@lukas-h lukas-h closed this as completed Jan 30, 2024
@lukas-h
Copy link
Member

lukas-h commented Feb 8, 2024

what I wrote was not entirely correct. I did some test implementations of the difference function that is using @mohammedX6's poly_bool_dart lib.
I don't like that we have to convert between our turf's Polygon and poly_bool's Polygon classes.

@mohammedX6 do you think we can use your library as a blueprint to implement the functionality as part of turf_dart without a dependency to your library?

I created a new issue for that: #166

CC @jsiedentop

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants