Skip to content
This repository has been archived by the owner on Apr 8, 2021. It is now read-only.

visualizing MATH410 fails #4

Open
tybug opened this issue Apr 7, 2021 · 1 comment
Open

visualizing MATH410 fails #4

tybug opened this issue Apr 7, 2021 · 1 comment

Comments

@tybug
Copy link

tybug commented Apr 7, 2021

image

MATH410's prereqs are:

Prerequisite: 1 course with a minimum grade of C- from (MATH240, MATH461, MATH341); and 1 course with a 
minimum grade of C- from (MATH340, MATH241); and minimum grade of C- in MATH310. Or students who 
have taken courses with comparable content may contact the department.

and when this repo tries to fetch MATH341 from umdio, umdio comes back to us and tells us that course doesn't exist, since it's not being offered this semester. It has, however, been offered in previous semesters (see: https://planetterp.com/course/MATH341).

Errors like this could either be ignored (so the graph at least still renders, albeit inaccurately since it doesn't have prereqs from MATH341), or previous semesters could be checked when umdio returns a 404. Or perhaps umdio should even be the one to return in what semesters a course has been offered, instead of only checking the current semester?

@konaraddi
Copy link
Owner

Thanks for bringing this up and creating an issue. Yeah this site is inaccurate and I wouldn't recommend solely relying on it when creating a multi-year plan for the courses you want to take. This project could handle errors by at least letting the user know what errors occurred. This project isn't maintained so feel free to clone or fork if you want to improve upon it!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants