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

HQ: Use reprojected access vector in rasterization task #1637

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

emilyanndavis
Copy link
Contributor

Description

Fixes #1615 by passing the reprojected access vector to the rasterize_access task

Checklist

  • Updated HISTORY.rst and link to any relevant issue (if these changes are user-facing)
  • [ ] Updated the user's guide (if needed)
  • Tested the Workbench UI (if relevant)

@emilyanndavis
Copy link
Contributor Author

@dcdenu4 I think this is the only change needed, but I noticed that create_access_raster_task and rasterize_access_task are being added to access_task_list, which allows them to later be passed to total_degradation_task as dependencies.

Do I need to add reproject_access_task to access_task_list?

Or is it sufficient that reproject_access_task is already a dependency of rasterize_access_task (and therefore implicitly understood as a dependency of total_degradation_task)?

Also wondering if you have any suggestions as to how to go about testing this, or if there might be a scenario in another test suite I can use as an example?

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 this pull request may close these issues.

HQ doesn't use reprojected access vector
1 participant