Open-EO / Open-EO/openeo-python-driver

future proof load_ml_model

Open
#306 1 comment 0 reactions 1 assignee View on GitHub

@JeroenVerstraelen is already working on this.

Since Sep 2, 2024.

Dominant language
Python
Stars
12
Forks
8
Avg merge
6d 3h
Merged PRs (30d)
4

Description

We have an old spec of load_ml_model in openeo-python-driver:

https://github.com/Open-EO/openeo-python-driver/blob/22f89cfaa30306115f5479bdfc8b9ea8ebd0f04e/openeo_driver/specs/openeo-processes/experimental/load_ml_model.json#L2-L37

this was based on an old version of load_ml_model in openeo-processes, which was removed by https://github.com/Open-EO/openeo-processes/pull/417

The new version of load_ml_model is under construction at https://github.com/Open-EO/openeo-processes/pull/441

Important detail: the single argument of old load_ml_model was named id (which will be weird because only urls will be passed in practice), while the new one is uri which makes more sense

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.