-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Support for spatialite in sqlite in osgeo/gdal:ubuntu-small-latest #5657
Comments
There is the "full" version if "small" is not enough. |
Yes, I'm aware of that. |
Do you think it would be better choice for the majority of the users of the ubuntu-small to include spatialite? Many dependencies that spatialite requires (proj, geos...) are already included so it might not be a big difference. The alpine-normal that includes Spatialite is 242 MB while ubuntu-small is 280 MB. It seems that geopandas somehow depends on osgeo/gdal:ubuntu-small-latest. It may not be a sustainable decision because I think that the GDAL project does not promise that the docker builds will remain the same. I wonder if the geopandas project could modify the build recipes and make an own dockerfile that is optimal for the project. For example by removing the proj gridshift files from the ubuntu-full the file size would go down from 1.5 GB to maybe 900 MB. |
Difficult to judge for me what the majority of users need, but because spatialite is already linked within gdal I supposed that adding this cannot have a large impact on the image size. For the geopandas/pyogrio project it is not strictly necessary that this is added, as the CI builds run both on a conda based image, where the gdal installation does support spatialite within sqlite, and on the osgeo/gdal:ubuntu-small-latest image. |
GeoPackage standard actually defines only 5 functions: ST_IsEmpty, ST_MinX, ST_MaxX, ST_MinY, and ST_MaxY but I agree that the functions coming from SpatiaLite are very usable extra. |
I noticed that the osgeo/gdal:ubuntu-small-latest does support sqlite vector drivers (as documented here), but when running a dialect="SQLITE" query on a native sqlite file (a Geopackage), it is not possible to use spatialite functions. Using dialect='INDIRECT_SQLITE' it is possible to use them, but with a significant performance impact.
Is it a conscious choice not to enable spatialite within sqlite?
The text was updated successfully, but these errors were encountered: