Skip to content
This repository was archived by the owner on Jul 11, 2025. It is now read-only.

feat: regional assessment job #65

Merged
merged 5 commits into from
May 16, 2025
Merged

Conversation

PeterBaker0
Copy link
Collaborator

Implements regional assessment job, and modularises some aspects which are common to both jobs.

To test, ensure you add the REGIONAL_ASSESSMENT to your .env job types if running a local worker, so that it will handle these jobs.

Signed-off-by: Peter Baker <peter.baker122@csiro.au>
…f COG writing status

Signed-off-by: Peter Baker <peter.baker122@csiro.au>
Signed-off-by: Peter Baker <peter.baker122@csiro.au>
Signed-off-by: Peter Baker <peter.baker122@csiro.au>
Comment on lines -348 to -350
@debug "Pulling out raster"
assessed = Raster(assessed_fn; missingval=0, lazy=true)

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ConnectedSystems realised this was doing extra work - hence removal - assessed is already defined in the case where we just computed it - needs to be in the cache case instead

Co-authored-by: Takuya Iwanaga <takuyai@gmail.com>
Copy link
Collaborator

@ConnectedSystems ConnectedSystems left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@PeterBaker0 PeterBaker0 merged commit 4b627f4 into main May 16, 2025
1 check failed
@ConnectedSystems ConnectedSystems deleted the feat/regional-assessment-job branch May 16, 2025 03:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants