Khalid Riyaz

Results 11 comments of 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