Skip to content

Check only existing processes are mapped #13

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

Open
sophieherrmann opened this issue Jul 20, 2021 · 1 comment
Open

Check only existing processes are mapped #13

sophieherrmann opened this issue Jul 20, 2021 · 1 comment

Comments

@sophieherrmann
Copy link
Collaborator

There should be some kind of check whether a created openeo-processes-python call actually exists and can be executed.

As a first version all process names supported in openeo-processes-python could be hard coded.

In a second step also the parameters should be checked.

https://github.com/Open-EO/openeo-processes-python

@danielFlemstrom
Copy link

danielFlemstrom commented May 31, 2022

Maybe this should rather be done in the process_discovery endpoint of OpenEO-API to avoid direct dependencies between the repositories? At RISE, we implemented the checkup something like shown below, maybe you want to cache the process-list (we do this in the calling function to avoid unnecessary http traffic).
IMHO, I think the proper way would be to implement something like get_implemented_process_defsin openeo-processes-python to avoid using dir on openeo_processes....

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants