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

Merge GET /processes and GET /processes/{process_id}? #54

Closed
m-mohr opened this issue Mar 11, 2018 · 1 comment
Closed

Merge GET /processes and GET /processes/{process_id}? #54

m-mohr opened this issue Mar 11, 2018 · 1 comment
Labels
Milestone

Comments

@m-mohr
Copy link
Member

m-mohr commented Mar 11, 2018

During development of the clients I found it very inconvenient that you need to call GET /processes/{process_id} for all processes over and over again. I'd suggest to merge both endpoints, which basically adds the args only:

Example for /processes:

[
  {
    "process_id":"NDVI",
    "description":"Computes the normalized difference vegetation index (NDVI) for all pixels of the input dataset.",
    "args":{
      "red":{
        "description":"red band ..."
      },
      "nir":{
        "description":"near infrared band ..."
      }
    }
  },
  {
    "process_id":"median_time",
    "description":"Applies median aggregation to pixel time series for all bands of the input dataset.",
    "args":{
      "A":{
        "description":"input product (time series)"
      }
    }
  }
]

GET /processes/{process_id} would be removed completely.

Not sure about GET /data and GET /data/{product_id}. The latter delivers more additional information to /data and I think datasets are not requested that often as only one or two datasets are usually used per request, but more than one or two processes per request.

@m-mohr m-mohr added this to the v0.1.0 milestone Mar 11, 2018
@m-mohr m-mohr changed the title Merge GET /processes and GET /processes/{process_id}? Merge GET /processes and GET /processes/{process_id}? (Also: /udf_runtimes/{lang}/{udf_type}) Mar 11, 2018
@m-mohr m-mohr changed the title Merge GET /processes and GET /processes/{process_id}? (Also: /udf_runtimes/{lang}/{udf_type}) Merge GET /processes and GET /processes/{process_id}? Mar 11, 2018
@m-mohr m-mohr added process discovery and profile discovery data discovery udfs and UDF runtime disctovery labels Mar 11, 2018
@m-mohr m-mohr modified the milestones: v0.3, v0.4 Mar 26, 2018
@m-mohr
Copy link
Member Author

m-mohr commented Apr 4, 2018

Merging processes would depend on how extensive #47 will be.
Merging data depends on the introduction of STAC (#64).

@m-mohr m-mohr removed the enhancement label Apr 5, 2018
@m-mohr m-mohr removed the udfs and UDF runtime disctovery label Jun 4, 2018
@m-mohr m-mohr modified the milestones: v0.4, v0.3 Jul 4, 2018
@m-mohr m-mohr closed this as completed Jul 5, 2018
m-mohr added a commit that referenced this issue Jul 5, 2018
…s been ported to the API spec with a proper OpenAPI schema. This should solve issues #47, #54, ...
soxofaan added a commit to Open-EO/openeo-python-client that referenced this issue Sep 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant