nusmodifications / nusmods

🏫 Official course planning platform for National University of Singapore.
https://nusmods.com
MIT License
578 stars 315 forks source link

Prerequisites tree for CS4243 broken #408

Closed acruis closed 6 years ago

acruis commented 7 years ago

Link to module page: https://nusmods.com/modules/CS4243

Red boxes indicating prerequisite modules within the prerequisite tree do not contain the module code:

screen shot 2017-07-07 at 4 31 50 pm

Reproduced on Chrome 59.0.3071.115 and Firefox 54.0.1.

yamgent commented 7 years ago

Graph seems to screw up when the prerequisites comprise of different possible combinations of modules (e.g. [X or Y or Z] and [A or B] and [C or [D or E]])

This also happens on CS3235 Computer Security:

cs3235

Dartteon commented 7 years ago

This is happening to CS4222 too (https://nusmods.com/modules/CS4222)

li-kai commented 6 years ago

Fixed in v3 with #451

acruis commented 6 years ago

👍