Why do we have a list `courses` that is filled with `(course, enrollment)` pairs when all we seem to use is the `course_id`s? Could we just have list of `enrollments` instead and use the `course_id` attribute in each enrollment? If we really do need the course objects, could you pls rename it to something that indicates it's a list of tuples as opposed to a list of course objects?