Set math 11's units to 5 units

ArturoAmaya / ExploratoryCurricularAnalytics

Scripts to convert UCSD Curriculum information from UCSD format into Curricular Analytics format.

1 stars 3 forks source link

Set math 11's units to 5 units #27

Closed SheepTester closed 2 years ago

SheepTester commented 2 years ago

Fixes https://github.com/ArturoAmaya/ExploratoryCurricularAnalytics/issues/20 by just overriding the units of MATH 11 exactly (i.e. not MATH 11 OR PSYC 60, which can be 4 units via PSYC 60, or Statistics Course (MATH 11 for GE), which is already 5 units in the plans)

SheepTester commented 2 years ago

(aside from the MATH 11 or PSYC 60 options, which we can leave as 4 units not 5).

Hmm then maybe the dictionary is too generalized for this. I think math 11 is probably the only oddity, anyways

SheepTester commented 2 years ago

Seems like other than MATH 18 there's only

[AN29 WA] MATH 11 OR PSYC 60

[GH25 RE] Statistics Course (MATH 11 FOR GE)
[GH26 RE] Statistics Course (MATH 11 for GE)

The latter two courses are already 5 units.

It's also worth noting that MATH 11 and PSYC 60 are in practice both offered

So I think just checking if the course string exactly matches MATH 11 should be enough

SheepTester commented 2 years ago

It seems like Revelle has a statistics requirement that math 11 satisfies, so that's probably how the Revelle advisors are familiar with math 11's unit count

SheepTester commented 2 years ago

PB27/ERC: "MATH 11" (5 units, edited from 4)

GH26/Revelle: "Statistics Course (MATH 11 for GE)" (5 units, unchanged)

AN29/Warren: "MATH 11 OR PSYC 60" (4 units, unchanged)