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.

Sign up for GitHub

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

Jump to bottom

[ENH] Rename intersection_polygon_polygon to intersect_polygons #337

Open
AlexanderJuestel opened this issue Jul 20, 2024 · 1 comment
Open

[ENH] Rename intersection_polygon_polygon to intersect_polygons #337

AlexanderJuestel opened this issue Jul 20, 2024 · 1 comment
Assignees
@AlexanderJuestel
Labels
enhancement New feature or request
Milestone
GemGIS 1.2

Comments

@AlexanderJuestel
Copy link
Collaborator

AlexanderJuestel commented Jul 20, 2024 •
edited
Loading

Describe the issue
As per pyOpenSci/software-submission#128 (comment), the function intersection_polygon_polygon and all following functions should be renamed to start with the verb intersect

This issue relates to #292
@AlexanderJuestel AlexanderJuestel mentioned this issue Jul 20, 2024
Open
24 tasks
@AlexanderJuestel AlexanderJuestel self-assigned this Jul 20, 2024
@AlexanderJuestel AlexanderJuestel added the enhancement New feature or request label Jul 20, 2024
@AlexanderJuestel AlexanderJuestel added this to the GemGIS 1.2 milestone Jul 20, 2024
AlexanderJuestel added a commit that referenced this issue Jul 20, 2024
@AlexanderJuestel
Fix #337
cbc9f31
@AlexanderJuestel
Copy link
Collaborator Author

This was addressed in cbc9f31

@AlexanderJuestel AlexanderJuestel mentioned this issue Jul 20, 2024
Open
30 tasks
AlexanderJuestel added a commit that referenced this issue Jul 20, 2024
@AlexanderJuestel
Fix tests #337
218341b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AlexanderJuestel AlexanderJuestel

Labels
enhancement New feature or request
Projects
None yet
Milestone
GemGIS 1.2
Development

No branches or pull requests
1 participant
@AlexanderJuestel