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

Extend and improve process graph repository #20

Open
1 of 5 tasks
christophfriedrich opened this issue Mar 26, 2019 · 3 comments
Open
1 of 5 tasks

Extend and improve process graph repository #20

christophfriedrich opened this issue Mar 26, 2019 · 3 comments

Comments

@christophfriedrich
Copy link
Collaborator

christophfriedrich commented Mar 26, 2019

There is a process graph repository, but it is very basic. It could be improved by the following things:

  • The Hub could also list all back-ends that support the processes in the process graph (idea by @m-mohr taken from Add process graph repository #2)
  • Add search feature for submitted process graphs -- once the repository contains more than a handful of entries, keeping them discoverable is important ✳️
  • Add user management to process graph exchange -- currently, the repository only allows the creation, but not the modification or deletion of submitted entries; and, maybe even more importantly, snippets are not attributed to their authors ✳️
  • Add endpoint that returns individual submitted process graphs by identifier -- this would allow users to seamlessly use snippets from the openEO Hub repository in their own process graphs via openEO's run_process_graph process ✳️
  • Make the process graph repository a 'pseudo-backend' by implementing the openEO API's /process_graphs endpoints -- this way, users could interact with the openEO Hub repository via the openEO-API-compatible tools they already know and use ✳️

list to be continued

✳️ = idea taken from my bachelor thesis

See #2 for the initial process graph repository idea.

@m-mohr m-mohr mentioned this issue Mar 28, 2019
@m-mohr m-mohr added this to the v1.0 milestone Aug 16, 2019
christophfriedrich added a commit that referenced this issue Sep 4, 2019
Compliant to openEO API v0.4.2.

The Hub now supports `GET /process_graphs`, `POST /process_graphs`, and `GET /process_graphs/{process_graph_id}`.

The Hub does not support  any authentication, i.e. *everybody* can submit process graphs and *everybody* can access them. (That's also the reason why the PATCH and DELETE endpoints are not supported, i.e. why point 5 of #20 is not yet addressed.)
@christophfriedrich
Copy link
Collaborator Author

Point 4 in the list above is essentially a sub-feature of point 5: GET /process_graphs/{process_graph_id} is one of the five /process_graphs endpoints.

Commit a6c2e5e made GET /process_graphs and POST /process_graphs openEO-API-compliant (v0.4.2); and in dd3d267 I added GET /process_graphs/{process_graph_id}.

The PATCH and DELETE endpoints shouldn't be supported before there is no user management, otherwise anyone could just remove all stored process graphs. I.e. point 5 depends on point 3.

➡️ Point 4 is done, point 5 is now supported 3/5 = 60%.

@m-mohr
Copy link
Member

m-mohr commented Sep 16, 2019

This issue is on hold and needs further investigation based on the outcomes of Open-EO/openeo-api#85 for process graphs.

@bgoesswe
Copy link
Member

bgoesswe commented Dec 5, 2019

Just to add:
I tried to use this feature for the QGIS plugin, but for better user experience, it would be nice to know on which backend the process graph was originally executed, just to know where it should work unchanged and where to get the info about the collection id used.

@m-mohr m-mohr modified the milestones: v1.0, v0.7 May 8, 2020
@m-mohr m-mohr modified the milestones: v0.7, future May 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants