Repository navigation
Add IERS-specific remote timeout - #5111
Conversation
| the local ``iers.conf.remote_timeout`` value. | ||
| """ | ||
| with utils.data.conf.set_temp('remote_timeout', conf.remote_timeout): | ||
| return utils.data.download_file(*args, **kwargs) |
There was a problem hiding this comment.
an alternative might be to do this instead of the with:
kwargs['timeout'] = conf.remote_timeout()
return utils.data.download_file(*args, **kwargs)
That should be the same, but it might be a bit less prone to trouble than set_temp? (Although maybe this isn't worth worrying about)
|
An alternative to my comment above: put the My main concern here is that someone else hacking on this might not understand why None of that is a big deal, so if you like it as is, @taldcroft, I'm 👍 to merge this. |
|
@eteq - I was on the fence about whether the name should be different, and even had My version using the |
Add IERS-specific remote timeout
Inspired by #5098 and #5099.
cc: @saimn @eteq @astrofrog