geopython / pygeoapi

pygeoapi is a Python server implementation of the OGC API suite of standards. The project emerged as part of the next generation OGC API efforts in 2018 and provides the capability for organizations to deploy a RESTful OGC API endpoint using OpenAPI, GeoJSON, and HTML. pygeoapi is open source and released under an MIT license.
https://pygeoapi.io
MIT License
474 stars 255 forks source link

Process execution request parsing not aligned with OGC API - Processes #1285

Closed ricardogsilva closed 4 weeks ago

ricardogsilva commented 1 year ago

Description When asking for the execution of a process, pygeoapi parses the incoming request and only keeps the inputs property of the request:

https://github.com/geopython/pygeoapi/blob/f436f0553f36213fb7be43f19a37d30c143c1c8a/pygeoapi/api.py#L3558

According to OGC API - Processes, an execution request has other relevant properties, such as:

In order to be inline with the spec, pygeoapi ought to pass all properties of the execution request down to the process manager and onto processes, and implement these alternative process execution features

totycro commented 1 year ago

We actually need the subscriber property in our company, so I'm going to look into this.

totycro commented 1 year ago

^ this PR is a straightforward implementation of subscriber, we can discuss if anything more advanced/different is needed

StefanBrand commented 11 months ago

@tomkralidis @ricardogsilva Please, comment on the status of this issue and the related merge request(s). We would like to send Slack messages for finished jobs.

ricardogsilva commented 10 months ago

@StefanBrand

To the best of my knowledge there hasn't been much progress on this.

I've reviewed the proposed PR back in July and I'm +1 for it to be merged (with some smaller tweaks, which have mostly already been addressed).

Personally I'm more interested in having support for outputs, as that would unlock a lot of customization in how processes are run - I'm also waiting on the pending PR to be merged so that I can work on implementation of outputs after it.

There could however be some deeper plan for this, as I am not fully in the loop of all the latest OGC-related developments. So I guess the status of this is that we are waiting on @tomkralidis input.

github-actions[bot] commented 5 months ago

As per RFC4, this Issue has been inactive for 90 days. In order to manage maintenance burden, it will be automatically closed in 7 days.

StefanBrand commented 5 months ago

Related PRs #1313 and #1448 were updated 5 days and 34 days ago, respectively.

github-actions[bot] commented 2 months ago

This Issue has been inactive for 90 days. As per RFC4, in order to manage maintenance burden, it will be automatically closed in 7 days.

StefanBrand commented 2 months ago

Current status if I'm not mistaken:

tomkralidis commented 4 weeks ago
* `response`: currently not implemented

Note that response is implemented. PR #1756 adds updates for supporting binary data when a process execution requests response: document. I've also updated the documentation with pygeoapi's OAProc implementation details/nuances for good measure (thanks to @ricardogsilva for the valuable discussion today to help clarify this).