You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Apr 16, 2023. It is now read-only.
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)
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.
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.
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
The code in
autofocus/predict
works, but it was written in a hurry and would probably benefit from some careful attention.The text was updated successfully, but these errors were encountered: