search

Open-EO / openeo-aggregator

openEO driver that combines multiple other drivers
https://open-eo.github.io/openeo-aggregator/
Apache License 2.0
1 stars 1 forks source link

random forest: providing training job with aggregator job id fails #70

Closed jdries closed 2 years ago

jdries commented 2 years ago

When referencing a random forest by job id, this doesn't work: training_job = c.job("vito-j-cc66a593416043468e4c5ca83ef90b28") it gives an error when trying to apply random forest.

This does work: training_job = c.job("j-cc66a593416043468e4c5ca83ef90b28")

probably the 'vito-' prefix needs to be stripped automatically.

soxofaan commented 2 years ago

FYI: the vito- prefix is added to the job id by the aggregator (to be able to determine which back-end a given job id belongs to).

If that job-id is used as reference for loading a ML model, the aggregator should strip it away again when forwarding to the upstream back-end. So I think this is an aggregator bug. For example, we had to do the same in load_result before: https://github.com/Open-EO/openeo-aggregator/issues/19

soxofaan commented 2 years ago

done