Penn Course Review Script

scores.py

import sys

import requests
import simplejson as json


class ReviewRetriever:

    def __init__(self, dept, key):
        self.key = key
        self.dept = dept
        self.professor_scores = {}

    # Retrieves an endpoint on the Penn Course Review API
    def get_data(self, endpoint):
        base_url = "http://api.penncoursereview.com/v1"
        url = base_url + endpoint + "?token=" + self.key
        r = requests.get(url)
        if hasattr(r, 'json'):
            return r.json
        else:
            return json.loads(r.content)

    # Retrieve a list of course reviews for a given department
    def get_courses(self):
        courses = self.get_data("/depts/" + self.dept)['result']['coursehistories']
        course_ids = [course['id'] for course in courses]
        return course_ids

    # Load the reviews for a given course id
    def load_reviews(self, course):
        reviews = self.get_data('/coursehistories/' + str(course) + '/reviews')['result']['values']
        for review in reviews:
            name = review['instructor']['name']
            rating = float(review['ratings']['rInstructorQuality'])
            if not (name in self.professor_scores):
                self.professor_scores[name] = []
            self.professor_scores[name].append(rating)

    # Return the average score for each professor
    def average_scores(self):
        return {prof: (sum(scores) / len(scores)) for prof, scores in self.professor_scores.items()}

    # Sort and print the list of professor averages
    def print_scores(self):
        self.professor_scores = {}
        course_ids = self.get_courses()
        for course_id in course_ids:
            self.load_reviews(course_id)
        scores = self.average_scores()
        sorted_scores = sorted(scores.items(), lambda x, y: cmp(y[1], x[1]))
        for prof, score in sorted_scores:
            print "%s %.2f" % (prof, score)


def main(dept, key):
    rr = ReviewRetriever(dept, key)
    rr.print_scores()

if __name__ == "__main__":
    if (len(sys.argv) < 2):
        print "usage: scores.py <department> <apikey>"
    else:
        main(*sys.argv[1:])

Probably no reason to update this, but if a reason ever arises check out the PCR wrapper I wrote. It ought to simplify a bit of the logic.

Actually, on second though, seems like a useful example. If you don't mind, I'd like to integrate into the project after updating it.

Please see my fork. It cuts the load time significantly (though it is apparent there is a need for threads inside of the penncoursereview library).