Khalid Riyaz
Khalid Riyaz
I would like to work on this one. @RishabhJain2018
@RishabhJain2018 Link to Google doc with a checklist of all the relevant unit tests to be written: https://docs.google.com/document/d/1IF3APjd93hUeov4KKRGDupMlcjpLO-MYo3_lEP8e42k/edit?usp=sharing
@RishabhJain2018 ..any comments/changes required to the doc?
@savish28 The code in `aws_utils` seems to have many levels of calls inside the delete function. Seems a bit redundant and maybe you can flatten this by having each function...
> > @savish28 The code in `aws_utils` seems to have many levels (aka recursive?) of calls inside the delete function. Seems a bit redundant and maybe you can flatten this...
Also, @savish28 you've tested this on local and UI/Error responses look fine? Can you add a screenshot/gif for the same?
@shiv6146 This may help for now, until tests for the submission worker are written. Please have a look: https://coverage.readthedocs.io/en/coverage-4.3.4/excluding.html
@nsjcorps What's the issue? Are you still unable to install Docker?
Goes with [EvalAI #2430](https://github.com/Cloud-CV/EvalAI/pull/2430). @RishabhJain2018 @Ram81 PTAL.
@RishabhJain2018 @deshraj Please review. Thanks! @isht3