Merge "Fix PEP 8 errors"

.gitignore

@@ -34,3 +34,9 @@ nosetests.xml
 .mr.developer.cfg
 .project
 .pydevproject
+
+# QA
+coverage.xml
+diff_*.html
+*.report
+report

Makefile

@@ -27,5 +27,5 @@ coverage: test
 	diff-quality --violations=pylint --html-report diff_quality_pylint.html
 
 	# Compute style violations
-	pep8 > pep8.report || echo "Not pep8 clean"
+	pep8 edx > pep8.report || echo "Not pep8 clean"
 	pylint -f parseable edx > pylint.report || echo "Not pylint clean"

edx/__init__.py

-

edx/analytics/__init__.py

-

edx/analytics/tasks/__init__.py

-

edx/analytics/tasks/course_enroll.py

@@ -115,8 +115,8 @@ class CourseEnrollmentEventsPerDayMixin(object):
                 enrollment_status = new_enrollment_status
             else:
                 logger.warning("WARNING: duplicate enrollment event {status} "
-                    "for user_id {user_id} in course {course_id} on {date}".format(
-                    status=new_enrollment_status, user_id=user_id, course_id=course_id, date=this_date))
+                               "for user_id {user_id} in course {course_id} on {date}".format(
+                                   status=new_enrollment_status, user_id=user_id, course_id=course_id, date=this_date))
 
             prev_date = this_date
 
@@ -144,7 +144,7 @@ class CourseEnrollmentChangesPerDayMixin(object):
 
         Args:
             key:  (course_id, datestamp) tuple
-            values:  iterator of enrollment_changes 
+            values:  iterator of enrollment_changes
 
             Input `enrollment_changes` are the enrollment changes on a day due to a specific user.
             Each user with a change has a separate input, either -1 (unenroll) or 1 (enroll).

edx/analytics/tasks/reports/enrollments.py

@@ -190,7 +190,7 @@ class EnrollmentsByWeek(luigi.Task):
 
         # List the dates of the last day of each week requested.
         start, weeks = self.date, self.weeks
-        days = [start - timedelta(i*7) for i in reversed(xrange(0, weeks))]
+        days = [start - timedelta(i * 7) for i in reversed(xrange(0, weeks))]
 
         # Sample the cumulative data on the requested days.
         # Result is NaN if there is no data available for that date.