lookahead icon indicating copy to clipboard operation
lookahead copied to clipboard

Semester 1 (Early-Start) and Semester 1 (Extended) subjects are not included.

Open manu354 opened this issue 4 years ago • 4 comments

Just realized this working on subjectListScraper.ts, is it desired to have these subjects included?

If it is desired I will make a PR updating the scraper functionality to include these.

Edit: same for semester 2

manu354 avatar Jan 25 '21 06:01 manu354

Sure, the more periods covered the better, I guess! This will also entail front-end work to add it to the drop down list. Let me know if you need some guidance.

Trontor avatar Jan 25 '21 10:01 Trontor

Will do :)

manu354 avatar Jan 25 '21 10:01 manu354

Hm actually, are you sure the premise is correct?

https://github.com/Trontor/lookahead/blob/master/subject-utils/scripts/subjectListScraper.ts#L118-L126

Seems like I've grouped them all under Semester 1/2

Trontor avatar Jan 25 '21 12:01 Trontor

If you look at subjects-lists (before #11 was merged) there are 2099 subjects in subjects_2021_semester_1.json

If you look on the handbook at how many subjects there are with semester 1 including early and extended starts, there are 2421 subjects!

When you select only Semester 1 it drops down to 2078 subjects, however, that's still not the 2099 that the previous json has -- not sure where that discrepancy of 21 is coming from. But I am pretty sure that the larger discrepenacy is from not including early and extended start subjects.

manu354 avatar Jan 25 '21 13:01 manu354