autofocus icon indicating copy to clipboard operation
autofocus copied to clipboard

Clean up app code

Open gsganden opened this issue 6 years ago • 3 comments

The code in autofocus/predict works, but it was written in a hurry and would probably benefit from some careful attention.

gsganden avatar Aug 04 '19 00:08 gsganden

Hey, I have tried to clean up and organize the code in predict. I would like to open a new branch and push to it, but it seems that I do not have the permission to do that.

I've also managed to find and close some bugs:

  • The zip is removed from the server after completion
  • The complete zip is now analyzed (before it returned within the loop)

DigitalPhilosopher avatar Oct 15 '19 17:10 DigitalPhilosopher

Awesome!

Sorry, the PR process needs a bit of cleanup and documentation. Please create a fork and submit your PR against the dev branch. LMK if you run into any issues.

gsganden avatar Oct 15 '19 22:10 gsganden

Perfect, thank you. I created the PR now. After a few changes the build is now running on travis.

One side note: Since there was the bug in the loop at the predict_zip endpoint, the running PR takes all files instead or running a loop. I think the better solution here would be to add a method in ZipFile named apply, which can apply a method on all the files in the zip file.

DigitalPhilosopher avatar Oct 16 '19 08:10 DigitalPhilosopher