-
Notifications
You must be signed in to change notification settings - Fork 5
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
jgrss/v170 issues #69
Conversation
* format * remove attr * add grid attr check * pin upper lightning * change * change * format * docs: Update README.md (#62) minor update to install of pyenv * make the default resolution None * fix the bounds setting --------- Co-authored-by: Michael Mann <mmann1123@gmail.com>
if rechunk: | ||
src = src.chunk( | ||
chunks={ | ||
'band': -1, | ||
'y': new_row_chunks, | ||
'x': new_col_chunks, |
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.
oh nice! This adds another check when running inference on small images 👍
ref_res: T.Optional[float] = 10.0, | ||
ref_res: T.Optional[T.Union[float, T.Tuple[float, float]]] = 10.0, |
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.
so here you also accept tuples of values?
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.
@nnguyen622 the default cell size is now None
(see the commit in args.yml
). This change allows the resolution to be gathered from the image (e.g., in case the user doesn't know the exact cell size). If taken from the image, and in cases of non-square pixels, tuples are allowed.
@@ -85,7 +85,7 @@ shared_create: | |||
long: res | |||
help: The cell resolution | |||
kwargs: | |||
default: 10.0 | |||
default: !!null |
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.
If the default here is unchanged by the user, the cell resolution will be taken directly from the image.
@@ -739,7 +739,14 @@ def create_datasets(args): | |||
) | |||
|
|||
total_iters = len( | |||
list(itertools.product(inputs.year_lists, inputs.regions_lists)) | |||
list( |
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.
This chunk fixes the broken progress bar.
From PR #64: