Commit 8f3ecfc1 by Tyler Hallada

Don't merge audit & honor. Add credit track.

parent b165ff08
AUDIT = u'audit' AUDIT = u'audit'
CREDIT = u'credit'
HONOR = u'honor' HONOR = u'honor'
PROFESSIONAL = u'professional' PROFESSIONAL = u'professional'
PROFESSIONAL_NO_ID = u'no-id-professional' PROFESSIONAL_NO_ID = u'no-id-professional'
VERIFIED = u'verified' VERIFIED = u'verified'
ALL = [AUDIT, HONOR, PROFESSIONAL, PROFESSIONAL_NO_ID, VERIFIED] ALL = [AUDIT, CREDIT, HONOR, PROFESSIONAL, PROFESSIONAL_NO_ID, VERIFIED]
...@@ -70,9 +70,10 @@ class Command(BaseCommand): ...@@ -70,9 +70,10 @@ class Command(BaseCommand):
enrollment_mode_ratios = { enrollment_mode_ratios = {
'audit': 0.15, 'audit': 0.15,
'honor': 0.35, 'credit': 0.15,
'honor': 0.25,
'professional': 0.10, 'professional': 0.10,
'verified': 0.40 'verified': 0.35
} }
# Generate birth year ratios # Generate birth year ratios
...@@ -255,7 +256,7 @@ class Command(BaseCommand): ...@@ -255,7 +256,7 @@ class Command(BaseCommand):
username = options['username'] username = options['username']
video_id = '0fac49ba' video_id = '0fac49ba'
video_module_id = 'i4x-edX-DemoX-video-5c90cffecd9b48b188cbfea176bf7fe9' video_module_id = 'i4x-edX-DemoX-video-5c90cffecd9b48b188cbfea176bf7fe9'
start_date = datetime.datetime(year=2015, month=1, day=1, tzinfo=timezone.utc) start_date = datetime.datetime(year=2016, month=1, day=1, tzinfo=timezone.utc)
num_weeks = options['num_weeks'] num_weeks = options['num_weeks']
if num_weeks: if num_weeks:
......
...@@ -11,7 +11,8 @@ from analytics_data_api.v0 import models ...@@ -11,7 +11,8 @@ from analytics_data_api.v0 import models
# Below are the enrollment modes supported by this API. The audit and honor enrollment modes are merged into honor. # Below are the enrollment modes supported by this API. The audit and honor enrollment modes are merged into honor.
ENROLLMENT_MODES = [enrollment_modes.HONOR, enrollment_modes.PROFESSIONAL, enrollment_modes.VERIFIED] ENROLLMENT_MODES = [enrollment_modes.AUDIT, enrollment_modes.CREDIT, enrollment_modes.HONOR,
enrollment_modes.PROFESSIONAL, enrollment_modes.VERIFIED]
class CourseActivityByWeekSerializer(serializers.ModelSerializer): class CourseActivityByWeekSerializer(serializers.ModelSerializer):
......
...@@ -410,10 +410,6 @@ class CourseEnrollmentModeViewTests(CourseEnrollmentViewTestCaseMixin, DefaultFi ...@@ -410,10 +410,6 @@ class CourseEnrollmentModeViewTests(CourseEnrollmentViewTestCaseMixin, DefaultFi
G(self.model, course_id=course_id, date=self.date, mode=mode) G(self.model, course_id=course_id, date=self.date, mode=mode)
def serialize_enrollment(self, enrollment): def serialize_enrollment(self, enrollment):
# Treat audit as honor
if enrollment.mode is enrollment_modes.AUDIT:
enrollment.mode = enrollment_modes.HONOR
return { return {
u'course_id': enrollment.course_id, u'course_id': enrollment.course_id,
u'date': enrollment.date.strftime(settings.DATE_FORMAT), u'date': enrollment.date.strftime(settings.DATE_FORMAT),
...@@ -432,10 +428,6 @@ class CourseEnrollmentModeViewTests(CourseEnrollmentViewTestCaseMixin, DefaultFi ...@@ -432,10 +428,6 @@ class CourseEnrollmentModeViewTests(CourseEnrollmentViewTestCaseMixin, DefaultFi
cumulative += ce.cumulative_count cumulative += ce.cumulative_count
response[ce.mode] = ce.count response[ce.mode] = ce.count
# Merge the honor and audit modes
response[enrollment_modes.HONOR] += response[enrollment_modes.AUDIT]
del response[enrollment_modes.AUDIT]
response[enrollment_modes.PROFESSIONAL] += response[enrollment_modes.PROFESSIONAL_NO_ID] response[enrollment_modes.PROFESSIONAL] += response[enrollment_modes.PROFESSIONAL_NO_ID]
del response[enrollment_modes.PROFESSIONAL_NO_ID] del response[enrollment_modes.PROFESSIONAL_NO_ID]
...@@ -453,7 +445,6 @@ class CourseEnrollmentModeViewTests(CourseEnrollmentViewTestCaseMixin, DefaultFi ...@@ -453,7 +445,6 @@ class CourseEnrollmentModeViewTests(CourseEnrollmentViewTestCaseMixin, DefaultFi
# Create the expected data # Create the expected data
modes = list(enrollment_modes.ALL) modes = list(enrollment_modes.ALL)
modes.remove(enrollment_modes.AUDIT)
modes.remove(enrollment_modes.PROFESSIONAL_NO_ID) modes.remove(enrollment_modes.PROFESSIONAL_NO_ID)
expected = {} expected = {}
......
...@@ -513,9 +513,6 @@ class CourseEnrollmentModeView(BaseCourseEnrollmentView): ...@@ -513,9 +513,6 @@ class CourseEnrollmentModeView(BaseCourseEnrollmentView):
total += enrollment.count total += enrollment.count
cumulative_total += enrollment.cumulative_count cumulative_total += enrollment.cumulative_count
# Merge audit and honor
item[enrollment_modes.HONOR] = item.get(enrollment_modes.HONOR, 0) + item.pop(enrollment_modes.AUDIT, 0)
# Merge professional with non verified professional # Merge professional with non verified professional
item[enrollment_modes.PROFESSIONAL] = \ item[enrollment_modes.PROFESSIONAL] = \
item.get(enrollment_modes.PROFESSIONAL, 0) + item.pop(enrollment_modes.PROFESSIONAL_NO_ID, 0) item.get(enrollment_modes.PROFESSIONAL, 0) + item.pop(enrollment_modes.PROFESSIONAL_NO_ID, 0)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment