You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We found the pqdm library is the thing swallowing all the error messages. pqdm has an exception_behavior option, which defaults to "ignore", surprisingly. The other two settings, "deferred" and "immediate", are both good options for earthaccess.
@chuckwondo and I agree "immediate" is the most intuitive default, and we can establish a new flag fail_fast: bool = True to override it to "deferred" by passing download(..., fail_fast=False). Prior discussion in #36. Please continue discussion here :)
I feel exposing the "ignore" behavior isn't necessary because the user can do their own exception handling with try.
Thoughts?
The text was updated successfully, but these errors were encountered:
We found the
pqdm
library is the thing swallowing all the error messages.pqdm
has anexception_behavior
option, which defaults to"ignore"
, surprisingly. The other two settings,"deferred"
and"immediate"
, are both good options for earthaccess.@chuckwondo and I agree
"immediate"
is the most intuitive default, and we can establish a new flagfail_fast: bool = True
to override it to"deferred"
by passingdownload(..., fail_fast=False)
. Prior discussion in #36. Please continue discussion here :)I feel exposing the
"ignore"
behavior isn't necessary because the user can do their own exception handling withtry
.Thoughts?
The text was updated successfully, but these errors were encountered: