-
Notifications
You must be signed in to change notification settings - Fork 50
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
dask_GDL #575
base: develop
Are you sure you want to change the base?
dask_GDL #575
Conversation
…arning into geo_deep_learning_dask
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you think an RGBN test can be useful?
@@ -125,6 +125,6 @@ def main(params:Union[DictConfig, Dict]): | |||
# LOOP THROUGH LIST OF INPUT IMAGES | |||
for aoi in tqdm(list_aois, desc='Inferring from images', position=0, leave=True): | |||
logging.info(f'\nReading image: {aoi.aoi_id}') | |||
raster = aoi.raster | |||
geo_inference(raster, tiff_name=aoi.aoi_id, patch_size=chunk_size) | |||
geo_inference(tiff_image=str(aoi.raster_name), patch_size=chunk_size) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Geo_inference call does not have the argument tiff_image
but rather inference_input
https://github.com/NRCan/geo-inference/blob/main/geo_inference/geo_inference.py#L103
Thank you for submitting your PR and helping us with GDL.
Please read the template below, fill it out as appropriate, and make additional changes to your code as needed.
Please feel free to submit your PR even if it doesn't satisfy all of the requirements below.
If you need assistance or review from a maintainer, add the label Status: Help Needed or Status: Review Needed respectively.
After review, a maintainer will add the label Status: Revision Needed if further work is required for the PR to be merged.
Description
This PR intends to include dask for image contrast enhancement to speed up the pre-processing step.
In addition, it has removed the download step of reading stack data and instead it is streaming data directly to arrays.
Dependencies are:
Type of change
What types of changes does your code introduce? Put an
x
in all the boxes that apply:How Has This Been Tested?
Testing part is an ongoing step.
Checklist:
If you're unsure about any of these, don't hesitate to ask. We're here to help!
If your PR does not fulfill all of the requirements in the checklist above, that's OK! If you need help, @-mention a maintainer and/or add the Status: Help Needed label.