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

Implement Job splitter and manager in GFMAP #12

Closed
GriffinBabe opened this issue Jan 18, 2024 · 10 comments
Closed

Implement Job splitter and manager in GFMAP #12

GriffinBabe opened this issue Jan 18, 2024 · 10 comments
Assignees
Labels
enhancement New feature or request

Comments

@GriffinBabe
Copy link
Collaborator

GriffinBabe commented Jan 18, 2024

Implement a job splitter and a job manager in OpenEO GFMAP:

  • The Job Splitter takes in parameter a dataset containing polygons/points extraction location and dates. It first separates the job by the temporal-extent and then by the spatial-extent (by default, the spatial split is performed using the s2-grid, to avoid launching jobs that are too large on OpenEO)
  • The Job Manager launches jobs and keep track of their progress. If a job fails, logs are saved to a location and the user will be notified of the location of those logs. If the job is successful, the products are downloaded and a post-job script is run locally (if the user specifies it).

Maybe, if too many OpenEO jobs fail then cancel all the other jobs and abort the task.

@GriffinBabe GriffinBabe self-assigned this Jan 18, 2024
@GriffinBabe GriffinBabe added the enhancement New feature or request label Jan 18, 2024
@kvantricht
Copy link
Collaborator

@VincentVerelst you can find an example of dataset chunking here.

@kvantricht
Copy link
Collaborator

kvantricht commented Feb 1, 2024

@GriffinBabe
Copy link
Collaborator Author

@kvantricht From the dataset provided in issue #40 here is a first result from using the h3 library. Seems pretty good to me except maybe for the hexagons including a few points

Image

If I select only the points with extract=True field

Image

@GriffinBabe
Copy link
Collaborator Author

GriffinBabe commented Feb 5, 2024

@soxofaan I would also be interested in your opinion on this method as you might be more experienced with the technical limitations of OpenEO. Each hexagon here has an approximative 30km x 30km diameter

@GriffinBabe
Copy link
Collaborator Author

@kvantricht can I use the dataset you provided me as a unit test resource?

@kvantricht
Copy link
Collaborator

Looks pretty good indeed. I don't think the ones with just few points are a real problem at the moment? Might be a bit of overhead but seems not a priority to address that, do you think?

@kvantricht
Copy link
Collaborator

@kvantricht can I use the dataset you provided me as a unit test resource?

in principle yes but isn't it a bit big for that?

@soxofaan
Copy link
Member

soxofaan commented Feb 5, 2024

@soxofaan I would also be interested in your opinion on this method as you might be more experienced with the technical limitations of OpenEO. Each hexagon here has an approximative 30km x 30km diameter

That looks fine, I don't expect a problem here

@GriffinBabe
Copy link
Collaborator Author

Good
For the unit tests I will upload a subsampled version of it @kvantricht

@GriffinBabe GriffinBabe mentioned this issue Feb 12, 2024
1 task
@GriffinBabe
Copy link
Collaborator Author

Implemented and about to be merged #45

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants