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 per GeoPackage Table Frontend #970

Merged
merged 8 commits into from
Nov 5, 2024

Conversation

signedav
Copy link
Member

@signedav signedav commented Oct 21, 2024

The setting for gpkgMultiGeomPerTable on Schema Import

Screenshot from 2024-10-30 22-06-22

Otherwise the setting is invisible and the parameter is false

This does only work with a sufficient GDAL version * Otherwise it's invisible and false.

Generating QGIS Project

Reading such a project does only work with a sufficient GDAL version * If the schema has been created with the gpkgMultiGeomPerTable and it contains tables with multiple geometry columns, but the current GDAL version is not sufficient, we block the creation of a QGIS Project.

Screenshot from 2024-10-30 22-08-58

When we have a sufficient GDAL version * we can create a QGIS project, but need to warn, that others reading this project might have trouble.

image

GDAL Version 3.8

This functionality is not standardized and does only work with GDAL Version >= 3.8

Windows Builds have it since 3.36 but Ubuntu from the current repositories not yet.

When a schema is created with this parameter gpkgMultiGeomPerTable and the Model contains multiple geometries per table, it can neither create a project on QGIS with GDAL<3.8 nor read a project that has been created on a QGIS with GDAL >= 3.8.

If no multiple geometries per table, it still can create a project from it by GDAL<3.8 or open such a project.

Night mode:

image

Requires opengisch/QgisModelBakerLibrary#108

Resolves #531

@signedav signedav changed the title Multiple Geometries on Geopackage ili2db parameter Conern multiple geometries per GeoPackage Table Frontend Oct 28, 2024
@signedav signedav changed the title Conern multiple geometries per GeoPackage Table Frontend Concern multiple geometries per GeoPackage Table Frontend Oct 28, 2024
@signedav
Copy link
Member Author

@gacarrillor I guess this will lead to conflicts with the mulitlanguage GUI, but i guess it's an easy one to resolve...

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.

Concerning the orange frames (and the tooltip):

image

I'd suggest that we remove the QGIS version from these messages or at least that we move it after we've shown the installed GDAL version (perhaps even removing the bold style for the QGIS version). In that way, it'd be much easier to compare which GDAL version is needed vs. which GDAL version is installed.

@signedav
Copy link
Member Author

Thanks @gacarrillor I considered that. I shortened and simplified it generally.

@signedav signedav merged commit 4b06337 into master Nov 5, 2024
7 checks passed
@signedav signedav deleted the multigeom branch November 5, 2024 09:04
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.

Trouble creating relations in GPKG to multigeometry tables
2 participants