Skip to content
This repository has been archived by the owner on May 20, 2019. It is now read-only.

don't ignore the ml_models dir, just the files inside of it #127

Merged
merged 1 commit into from Aug 15, 2013

Conversation

natea
Copy link
Contributor

@natea natea commented Aug 9, 2013

if the ml_models dir doesn't exist, then you get an error like this:

[2013-08-09 16:39:41,219: ERROR/MainProcess] Could not remove ml grading models!
Traceback (most recent call last):
  File "./controller/tasks.py", line 104, in expire_submissions_task
    expire_submissions.remove_old_model_files()
  File "./controller/expire_submissions.py", line 222, in remove_old_model_files
    onlyfiles = [ f for f in os.listdir(settings.ML_MODEL_PATH) if os.path.isfile(os.path.join(settings.ML_MODEL_PATH,f)) ]
OSError: [Errno 2] No such file or directory: 'ml_models/'

So we should just include this dir in the repo, so one doesn't have to remember to create it manually.

VikParuchuri added a commit that referenced this pull request Aug 15, 2013
don't ignore the ml_models dir, just the files inside of it
@VikParuchuri VikParuchuri merged commit 080ae37 into dev Aug 15, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants