-
Notifications
You must be signed in to change notification settings - Fork 12
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
Output format options #32
Comments
For me it's entirely acceptable to put the output parameters in the body. |
Yes, that's probably the better choice to have it in the body or query string, might also depend on the API endpoint. Query string for GET (download / queue), body for POST (execute)? Regarding the options we might want to get inspired by GDAL or even make a (mostly?) compatible interface regarding the options. Example (compatible to GDAL; comments are only present to explain the options in this example):
Example for PNG:
We would need to come up with our own definitions for formats not supported by GDAL, e.g. custom time series. Even if GDAL is not supported we should enforce implementations to use these conventions. Of course, they might not support every option or add custom options, depending how they handle files as they might not use GDAL at all. By the way: We don't want to "create" web services (wms, tms, wcs, ...) using this endpoint, do we? |
Added with PR #35. Additional documentation might be needed though. |
IMHO the definition of the output format makes more sense in the process graph itself.
|
Interesting idea, @mundialis-dev. Would you mind copying this into a separate issue? |
As mentioned by @jdries in a recent discussion, there is no way to specify additional output format options. Therefore it seems reasonable to move the output_format query parameter to the body or even include it in the process graph. We already have something similar for the
POST /services
endpoint with theargs
in the body.Examples for output format options could be:
Example for the process graph:
Example in the body:
Maybe it is also possible to add it to the query (depends on how extensive the options can be):
...?format=jpeg&compression_level=0.9
The text was updated successfully, but these errors were encountered: