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

Expose options Parameter in @turf/boolean-intersects to facilitate ignoreSelfIntersections #2633

Open
01100100 opened this issue Jun 26, 2024 · 0 comments

Comments

@01100100
Copy link
Contributor

The @turf/boolean-intersects function does not handle cases when the user wants to ignore self intersections.

@turf/boolean-intersects internally uses the lineIntersect function, which already supports an options parameter. However, this flexibility is not currently exposed in @turf/boolean-intersects.

To enhance the functionality and flexibility of @turf/boolean-intersects, I suggest to add a options parameter. This parameter should be passed through to the lineIntersect function, allowing users to leverage the existing capabilities of lineIntersect, including the ability to ignore self intersections or any other options that lineIntersect supports.

To ensure backward compatibility, the options parameter can be added with a default value that maintains the current behavior of @turf/boolean-intersects.

This approach ensures that existing codebases using @turf/boolean-intersects will not be affected by this change, while new and existing users gain the ability to customize the function's behavior to better suit their needs.

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

No branches or pull requests

2 participants