search

venkatesh-sivaraman / FireRoad-Android

Android version for the MIT course planner app.
MIT License
2 stars 0 forks source link

Prerequisite Search #3

Closed brianadouglas closed 4 years ago

brianadouglas commented 5 years ago

After clicking on 'View Subjects with [subject number] as prerequisite', the search incorrectly returns nothing in the cases of 8.01 and 18.01, among others.

liuric commented 4 years ago

The prereq search for some reason used the toString() (which resulted in "calculus") method instead of the rawValue ("gir:calc2") field, which was what was listed for the prereqs strings for each course. The separation of GIR courses that were equivalent and rest and lab courses didn't filter out lab courses. I pushed these fixes in the prereq-search-bugfix branch, but the relevance of the courses are probably not correct, resulting in a weird order for the search results.

venkatesh-sivaraman commented 4 years ago

Resolved by #6