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.

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

Concern multiple geometries in same table for geopackage #108

Merged
merged 13 commits into from
Oct 31, 2024
Merged

Conversation

signedav
Copy link
Member

@signedav signedav commented Sep 27, 2024

  • ili2db 5.1.1
  • Passing parameter gpkgMultiGeomPerTable
  • Building layer url including geometry-column if the layer source is used multiple times (because of multiple geometries in a table)
  • Function to retrieve information about table with multiple geometry columns

Functionality used by opengisch/QgisModelBaker#970 Find more (conceptional) information there.

  • Tests
    • Testmodel

@signedav signedav marked this pull request as draft September 27, 2024 09:13
@signedav
Copy link
Member Author

signedav commented Oct 18, 2024

Find current GDAL Version of Windows Builds
https://github.com/jef-n/OSGeo4W/tree/master/src/gdal/osgeo4w
Means we can check the QGIS version to decide if we do it or we dont...

Current Ubuntu Nightly still with 3.4 apparently. Depending on the repository there... maybe we can check the GDAL Version directly, but I don't know how at the moment. Something like:

  const QString gdalVersionCompiled { GDAL_RELEASE_NAME };
  const QString gdalVersionRunning { GDALVersionInfo( "RELEASE_NAME" ) };

but not sure how it's handled in QGIS Api.

Edit:

See this https://stackoverflow.com/questions/3233674/how-can-i-get-the-installed-gdal-ogr-version-from-python/3412259#3412259

@signedav signedav changed the title Concern multiple geometries in same layer for geopackage Concern multiple geometries in same table for geopackage Oct 28, 2024
@signedav signedav marked this pull request as ready for review October 28, 2024 07:28
Copy link
Collaborator

@gacarrillor gacarrillor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good stuff!

@signedav signedav merged commit 1029271 into main Oct 31, 2024
5 checks passed
@signedav signedav deleted the multigeom branch October 31, 2024 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants