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.

Sign up for GitHub

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

Jump to bottom

Add method to load the results of a previous job #566

Closed
VictorVerhaert opened this issue Jun 3, 2024 · 1 comment · Fixed by #572
Closed

Add method to load the results of a previous job #566

VictorVerhaert opened this issue Jun 3, 2024 · 1 comment · Fixed by #572
Assignees
@VictorVerhaert
Labels
feature request nice-to-have usability

Comments

@VictorVerhaert
Copy link
Contributor

VictorVerhaert commented Jun 3, 2024 •
edited
Loading

Add a method to the openeo python client to directly load the result of a previous job. The python client in turn fetches the stac of the previous job and add a load_stac with this link to the PG.

The method should receive either a Job object of a job-id as argument.
@VictorVerhaert VictorVerhaert self-assigned this Jun 3, 2024
@VictorVerhaert VictorVerhaert mentioned this issue Jun 3, 2024
Open
@soxofaan
Copy link
Member

soxofaan commented Jun 3, 2024 •
edited
Loading

suggestion: use the method name load_stac_from_job, which is self-explaining and easy to discover (e.g will automatically be suggested by autocomplete functionality if you start typing "load_stac")

VictorVerhaert added a commit that referenced this issue Jun 11, 2024
@VictorVerhaert
added tests and option to provide jobid
b09565f 
issue #566
@VictorVerhaert VictorVerhaert linked a pull request Jun 11, 2024 that will close this issue
Issue566 load previous job #572
Merged
@soxofaan soxofaan mentioned this issue Jun 11, 2024
Merged
VictorVerhaert added a commit that referenced this issue Jun 13, 2024
@VictorVerhaert
processed review comments
211caf9 
issue #566
VictorVerhaert added a commit that referenced this issue Jun 13, 2024
@VictorVerhaert
rewrote canonical_links based on review feedback
fc6e29f 
issue #566
soxofaan pushed a commit that referenced this issue Jun 13, 2024
@VictorVerhaert
Issue #566 add load_stac_from_job to load_stac from job results
6bd598f 
Squashed merge of PR #572
soxofaan added a commit that referenced this issue Jun 13, 2024
@soxofaan
fixup! Issue #566 add load_stac_from_job to load_stac from job re…
e22fca7 
…sults
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@VictorVerhaert VictorVerhaert

Labels
feature request nice-to-have usability
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Issue566 load previous job Open-EO/openeo-python-client
2 participants
@soxofaan @VictorVerhaert