-
Notifications
You must be signed in to change notification settings - Fork 157
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
Fix all unexecutable code examples I can find. #693
Fix all unexecutable code examples I can find. #693
Conversation
@@ -133,7 +133,8 @@ def _split_out_geometry_columns(self) -> Tuple: | |||
""" | |||
columns_mask = pd.Series(self.columns) | |||
geocolumn_mask = pd.Series( | |||
[isinstance(self[col], GeoSeries) for col in self.columns] | |||
[isinstance(self[col], GeoSeries) for col in self.columns], | |||
dtype="bool", |
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.
Pandas was issuing a warning on this line so I corrected that warning.
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.
Thank you Thomson! Fixing these old smelly bits is really important and really improves UX of our library.
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.
At some point we should introduce doctest to cuspatial.
@gpucibot merge |
Description
A few old issues that are tasked for
22.10
involve fixing documentation that is incorrect. I worked through the existing nightly docs and executed all of the code samples, fixing and improving anything that had incorrect outputs or inputs.closes #460
closes #63
Checklist