-
Notifications
You must be signed in to change notification settings - Fork 22
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
Add support for force_2d with use_arrow=True in read_dataframe #300
Add support for force_2d with use_arrow=True in read_dataframe #300
Conversation
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.
Should we add a note to the docstring for read_dataframe
stating that shapely is used to force 2D geometries after reading when use_arrow
is True
?
I think you could move use of shapely down to read_arrow
(i.e., make shapely a requirement to use force_2d
in read_arrow
) and conditionally attempt to import shapely if force_2d
is True
and raise an ImportError if it isn't present.
…_dataframe-for-force_2d-for-use_arrow=True
At first sight, I don't think it adds a lot of useful information to the user whether the conversion is done by shapely or by gdal?
Yes, it indeed seems possible. Do we want to keep the interface of |
I think we want to keep Given that, I assume it is easier to keep this handling on the |
Whoops, I forgot about the conversion from WKB and thought we were already using shapely objects in |
Thanks! |
reference #297
Once #304 is merged, an extra use_arrow check can be activated or the other way around.