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

export_to_workspace #467

Open
m-mohr opened this issue Oct 5, 2023 · 3 comments · May be fixed by #485
Open

export_to_workspace #467

m-mohr opened this issue Oct 5, 2023 · 3 comments · May be fixed by #485
Assignees
Milestone

Comments

@m-mohr
Copy link
Member

m-mohr commented Oct 5, 2023

Proposed Process ID: export_to_workspace

Context

For the new workspace API we want a way to have a process that explicitly allows to store a result in a dedicated workspace instead of in "the normal user space"

We thought about having it as part of save_result, but then you either mix the storage location with the file formats or you get a number of parameters that depend on each other in save_result, both not ideal.

Backends may not do the intermediate step to store in the normal user space and directly store in the workspace although both processes are present.

The process usually only makes sense in batch jobs, I think.

NOTE: This process may have implications on save_result!

Summary

Exports a result to a registered workspace

Description

...

Parameters

data (or so?)

One issue is that save_result returns a boolean, so we need to look into that. Maybe it should return something else that we can feed into export_to_workspace as data? Maybe a list of files? Or just a generic "result" subtype?

name

Name of the registered workspace.

Type: string
Subtype: workspace-name

Optional: no

folder (or prefix?)

Stores the batch job with the given prefix / to the given folder in the workspace.

Default to null, which uses a unique identifier as prefix (e.g. batch job id).
An empty string stores them in the "root folder".

Q: Should the process by default override files or not?

Type: string or null

Optional: yes, default: null

Return Value

boolean?

Categories (optional)

  • export

Links to additional resources (optional)

  • Link to workspace API

Details about exceptions mentioned in the description (optional)

  • FileExists
@m-mohr m-mohr added this to the 2.0.0 milestone Oct 5, 2023
@m-mohr
Copy link
Member Author

m-mohr commented Oct 5, 2023

cc @Open-EO/team-eodc @christophreimer

@m-mohr m-mohr self-assigned this Oct 5, 2023
@jdries
Copy link
Contributor

jdries commented Oct 5, 2023

Should we have most of the discussion in this issue: Open-EO/openeo-api#376
Because there we already collected a lot of thoughts and requirements. I'm also not really clear yet on what the new workspace API looks like.

@m-mohr
Copy link
Member Author

m-mohr commented Oct 5, 2023

User-generated Collections and user workspaces are not necessarily the same, so they deserve separate issues, I think.
I tried to collect relevant issues and links in https://github.com/openEOPlatform/architecture-docs/issues/215#issuecomment-1748497731 (not being the lead here though, just started with it an hour ago)

@m-mohr m-mohr linked a pull request Dec 8, 2023 that will close this issue
@m-mohr m-mohr linked a pull request Sep 9, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants