SheepTester-forks / CurricularAnalytics.py

CurricularAnalytics.py is a Python port of CurricularAnalytics.jl, a toolbox for studying and analyzing academic program curricula.
https://pypi.org/project/curricularanalytics/
GNU Affero General Public License v3.0
2 stars 3 forks source link

Fix writing of curriculum metrics to CSV #9

Closed will-hou closed 3 months ago

will-hou commented 3 months ago

Resolves #8

Also matches the formatting of the metrics file exported by curricular analytics more closely:

will-hou commented 3 months ago

Testing with the public VCU Public Computer Engineering curriculum

Left is file with metrics downloaded from Curricular Analytics. Right is file without metrics downloaded from Curricular analytics and processed with CurricularAnalytics.py

Only differences are the ordering of pre-reqs but I don't believe that is important (also out of scope for this PR)

image

SheepTester commented 3 months ago

I'm not sure if we should match Curricular Analytics exactly since unit counts at some universities aren't necessarily an integer. For example, UCSD's language classes are 2.5 units

will-hou commented 3 months ago

I'm not sure if we should match Curricular Analytics exactly since unit counts at some universities aren't necessarily an integer. For example, UCSD's language classes are 2.5 units

Ah didn't know that. I removed that change

will-hou commented 3 months ago

@SheepTester Are there any other items you want me to address before this can get merged? Thanks!