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

Added Line intersect and Line Segment functionalities #72

Merged
merged 7 commits into from
Nov 8, 2020

Conversation

omanges
Copy link
Owner

@omanges omanges commented Nov 8, 2020

Added Line intersect and Line Segment functionalities similar to turf.js and requested in #66

@omanges omanges requested a review from sackh November 8, 2020 07:43
@omanges omanges changed the title Added Line intersect functionality Added Line intersect and Line Segment functionalities Nov 8, 2020
@lgtm-com
Copy link

lgtm-com bot commented Nov 8, 2020

This pull request introduces 2 alerts when merging 5ac58af into c35d8a1 - view on LGTM.com

new alerts:

  • 2 for Unused import

@sackh
Copy link
Collaborator

sackh commented Nov 8, 2020

Add example Notebook
Add Documentation for new functionality in rst files.

tests/test_misc.py Show resolved Hide resolved
tests/test_misc.py Show resolved Hide resolved
turfpy/_compact.py Show resolved Hide resolved
turfpy/misc.py Outdated Show resolved Hide resolved
turfpy/misc.py Outdated Show resolved Hide resolved
turfpy/misc.py Show resolved Hide resolved
turfpy/misc.py Show resolved Hide resolved
turfpy/misc.py Show resolved Hide resolved
@omanges omanges requested a review from sackh November 8, 2020 14:25
@omanges omanges merged commit 56f8474 into master Nov 8, 2020
@omanges omanges deleted the line_intersect branch November 8, 2020 14:30
@lgtm-com
Copy link

lgtm-com bot commented Nov 8, 2020

This pull request introduces 3 alerts when merging 8e20715 into c35d8a1 - view on LGTM.com

new alerts:

  • 3 for Unused import

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