stanfordnmbl / opencap-api

Apache License 2.0
5 stars 6 forks source link

adding try catch to prevent download failure when missing results #115

Closed antoinefalisse closed 11 months ago

antoinefalisse commented 11 months ago

Download is failing when the API is trying to download files from S3 that are missing. This might happen for instance for a trial for which video upload failed. This PR adds try except statements, such that download proceeds even if some stuff are missing. I tested on dev after adding a trial with missing videos to a session. Failed before the fix and worked after the fix. FYI the issue happened 693 times...Pushing to mainhere, because there is a bunch of things on dev. Will test on main after we merge.