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

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

Closed
jdries opened this issue Sep 5, 2022 · 2 comments
Closed
Assignees

Comments

@jdries
Copy link
Contributor

jdries commented Sep 5, 2022

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.

@JeroenVerstraelen JeroenVerstraelen self-assigned this Sep 5, 2022
@soxofaan
Copy link
Member

soxofaan commented Sep 9, 2022

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: #19

@soxofaan soxofaan transferred this issue from Open-EO/openeo-geopyspark-driver Sep 9, 2022
@JeroenVerstraelen JeroenVerstraelen linked a pull request Sep 13, 2022 that will close this issue
@soxofaan
Copy link
Member

done

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

Successfully merging a pull request may close this issue.

3 participants