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
separate spatial_index function #364
Conversation
"millions_to_hundreds" sjoint, where we split data into chunks and don't want to generate a rtree.Index for each chunk once again
maybe it is better to pass index as **kvargs? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add a test for this?
@@ -24,9 +45,9 @@ def sjoin(left_df, right_df, how='inner', op='intersects', | |||
Suffix to apply to overlapping column names (left GeoDataFrame). | |||
rsuffix : string, default 'right' | |||
Suffix to apply to overlapping column names (right GeoDataFrame). | |||
tree_idx : rtree.index for right GeoDataFrame, if None, will be created. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you put the explanation on a next line (see how it is done for the other kwargs). Typically it should have the form of (following the numpy docstring standard)
kwarg : type
Explanation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will do!
Thanks! I will try to think over a test |
separate spatial_index function might be of use for "millions_to_hundreds" sjoint, where we split data into chunks and don't want to generate a rtree.Index for each chunk once again