search

Cloud-CV / CloudCV-Old

CloudCV - Large-Scale Distributed Computer Vision As A Cloud Service
http://cloudcv.org
MIT License
51 stars 37 forks source link

PEP8 Fixes #19

Closed deshraj closed 8 years ago

deshraj commented 8 years ago

@dexter1691 I have fixed all the PEP8 issues. Please review the PR. Still some of the errors need to be fixed. Can you look into these:

./app/executable/vqa_files/tools.py:7:24: F821 undefined name 'svm' ./app/executable/vqa_files/tools.py:19:20: F821 undefined name 'svm' ./app/executable/vqa_files/tools.py:36:14: F821 undefined name 'KMeans' ./app/executable/vqa_files/tools.py:38:11: F821 undefined name 'euclidean_distances' ./app/executable/poi_files/svmutil.py:211:13: F811 redefinition of unused 'info' from line 200 ./celeryTasks/apiTasks/tasks.py:293:17: F821 undefined name 'mlab_obj' ./celeryTasks/webTasks/trainTask.py:181:28: F821 undefined name 'caffe_classify_image' ./celeryTasks/webTasks/poi_files/svmutil.py:207:13: F811 redefinition of unused 'info' from line 196 ./app/executable/vqa_files/driver.py:222:13: F821 undefined name 'Word2Vec'

As soon we get these errors fixed, we can start writing unit tests.

dexter1691 commented 8 years ago

Will fix the above mentioned issues in a separate PR.

deshraj commented 8 years ago

@dexter1691 ok :+1: