search

dandi / dandi-api-webshots-tools

0 stars 0 forks source link

Resolve timeouts; only restart subprocess on error #11

Closed jwodder closed 2 years ago

jwodder commented 2 years ago

Closes #10.

yarikoptic commented 2 years ago

if that gets us at least a little closer -- should we undraft and merge?

jwodder commented 2 years ago

@yarikoptic I don't know about "closer", considering that the script currently aborts upon hitting GitHub's secondary rate limit, which means that it's not even completing successfully.

yarikoptic commented 2 years ago

Looks good, thank you @jwodder ! Let's proceed. I will update submodule on drogon