morinted/schedule-generator

'schedule' div should be treated as a section, not a semester

SheikhIV opened this issue · 3 comments

The schedule generator works great but the CHM1311 labs are not showing up as of yet.

Thanks again!!

The uOttawa timetables site changed the contents of a particular divider to represent sections instead of semesters. The section ID should only be retrieved once per 'schedule' div, not for each tr. For CHM1311 all the labs have section 'Z', so right now they're going into a made-up section instead of being available for each lecture section:

capture d ecran 2018-05-19 a 20 16 27

Partially addressed in f42f7a4, but need to fix activity numbers only being captured as single digits.

Fixed the activity numbers in 2fffb81.

Note that CHM1311 is a bit strange since the lab sections all have one lab day that's shorter than the others, but all the labs in a set have the same section number. This causes duplicate schedules for each lab section, one with a short lab, one with a long one. I'm not sure how this would be addressed so I'm closing the issue, but just something to keep in mind.

Thanks for reporting this!