ArturoAmaya / ExploratoryCurricularAnalytics

Scripts to convert UCSD Curriculum information from UCSD format into Curricular Analytics format.
1 stars 3 forks source link

Math 18 prerequisite #37

Closed SheepTester closed 2 years ago

SheepTester commented 2 years ago

Discovered in #30

Because we assume everyone already has pre-calc credit, math 18 doesn't have any actual prerequisites. However, prereqs.csv does (correctly) list math 20a as a possible prerequisite for math 18, so if they show up in the same quarter, the generator will wrongly say that math 20a is a prereq for math 18.

I think the best course of action here is to add code that just ignores any prerequisites for math 18:

We might also want to add a check for any prerequisites that show up in a quarter in or after a course

ArturoAmaya commented 2 years ago

Yup! I agree that removing prereqs from MATH 18 makes sense. I just think it would be easier if we edit the prereqs doc we have. In the meantime, I'll flag it, but I won't delete it just yet.