-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[Processing] GRASS Support multiple vector output file formats #5726
Conversation
Nice |
For consistency, maybe place geopackage and shape file formats on top, followed by an alphabetical list of additional formats? That's what we now do elsewhere. |
Nice, it is very important to avoid v.external being the default import method in favor of v.in.ogr (which got improved again in GRASS GIS 7.4.0) |
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.
Looks good to me.
@@ -193,6 +223,48 @@ def getSupportedOutputRasterExtensions(): | |||
allexts.append(ext) | |||
return allexts | |||
|
|||
@staticmethod | |||
def getSupportedVectors(): |
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.
Same as above. If I'm not wrong QgsVectorFileWriter
already has methods for getting such information
vector = False | ||
raster = False | ||
# Is it an OGR driver? | ||
if (gdal.DCAP_VECTOR in metadata |
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.
Maybe better to rely on the core API here, e.g. supportedFiltersAndFormats() and other methods of the QgsRasterFileWriter class?
@medspx can you rebase? I will be happy to merge it after this |
Superseded by #6996 |
Description
Add multiple vector output file formats with v.out.ogr:
New formats:
OGR output options:
Checklist
fixes #11111
in the commit message next to the description[FEATURE]
in the commit message[needs-docs]
in the commit message and containt sufficient information in the commit message to be documentedscripts/prepare-commit.sh
script before each commit