Wrong course id for modelthinking

[acoomans]

./coursera list-course-ids command gives "modelthinker" as id; should be "modelthinking"

[fvieira]

Thanks, I hadn't noticed it. The problem is that the way I get that list is a bit hacky, basically I just go to https://www.coursera.org/landing/hub.php, get all links for the courses over there and take the part before the -class and assume it is the right id for that class. It works for most classes but there are some problems:

• computer vision is listed although it hasn't started. I couldn't find a way to distinguish which classes had started in that page
• game theory's url is http://www.game-theory-class.org/ while the id is gametheory (without the dash). I just removed all dashes from the ids and hoped there would never be a class id with a dash in it.
• and now I find that modelthinking's url is http://www.modelthinker-class.org/ which breaks my assumption. I'm not sure if this was always the case as I thought I had tested all ids at the time but whatever the case it is wrong now.

So, there are two solutions to this:

• Find a more reliable way to get the ids. If someone has an idea how to do it I can try to implement it.
• Make a special case for modelthinking hard-coded in the code, which is just adding another hack in top of the hacky code.

If nobody sugests a way to make the command more reliable I'll add the hack, but I'll wait a bit to see whether some suggestion comes.

Again, thanks for reporting it!

[acoomans]

Hey Francisco,

Here's a suggestion: from the landing page, go to each course by following its link (ie http://www.game-theory-class.org/). You'll be redirected to a coursera page like https://class.coursera.org/gametheory/auth/welcome (if you're not logged in yet) which contains the real course ID (ie gametheory)

In the case of not-yet-started courses, like computer vision, you are not redirected to a https://class.coursera.org/ link.

Hope it helps! -- AC

[fvieira]

Yes, that might work, although it will make the command run much slower since it will require a request for each class. I'll try to implement it as soon as I have the time and we'll see how fast I can make it run.

[fvieira]

Ok, I did exactly what you proposed, and with the addition of some threading through the use of the multiprocessing module, it is nearly as fast as it was before. Thanks for the sugestion!

[acoomans]

Nice! Good work!