ogc api processes - outputs #1420 #1435
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add support for OGC API Processes Outputs
Operation of add_job() and update_job() are called subject to the condition the configuration parameter connection is provided. Improved the error description returned in case of errors during the processing. Additional minor changes: -) improved returned message if an error is found during the parsing of the json input string -) amended returned message from echoProcessor in case of wrong input parameter
|
Made an error and mixed two PR. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview
Add support for OGC API Processes Outputs
The parameter
outputsis passed to the manager, which in turn pass it to the Processor.By default the processor returns all the outputs defined into the section
outputsofPROCESS_METADATA.If the processor receives the parameter
outputs(a dictionary), then only the outputs with the keys present into the parameter should be returned.Related Issue / Discussion
Solve #1420,
Subset of #1285
Similar issue of the same subset #1313
Additional Information
The change would be a breaking change,
but it would be fairly simple for maintainers to accept the parameters and ignore them for now if not interested,
as shown in the modified
HelloWorldProcessor.Contributions and Licensing
(as per https://github.com/geopython/pygeoapi/blob/master/CONTRIBUTING.md#contributions-and-licensing)