-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Curricular Analytics only draws one prereq #59
Comments
PS25/ERC has two MMW 14's It's probably because there are two MMW 14/GE, so one of them should be MMW 14 and the other is GE, but since the code removes /GE from course titles, it looks like there are two MMW 14s |
SE27, SE 131A has no prereqs The CSV doesn't recognize any of the courses as a prereq That's because prereqs.csv doesn't have any prereqs for SE 131A |
In US26, it mentions Sixth practicum This is wacky because I thought we were using Marshall for curricula, and Marshall doesn't say Sixth practicum
oh. oops, I forgot to break out of this loop, so it was using the most weird college to base the curricula, Sixth, not Marshall. oh well! |
I wonder if it has to do with the parent curriculum affecting the prerequisites of the degree plans |
Don't the parent versions of the courses not have pre-reqs at all? |
The GUI had absolutely no problem letting me add the pre-req relationship. It gets weirder. I downloaded the file that worked from the GUI and it's identical to the one I uploaded. Uh actually, I dunno if it updates the csvs you can dowload lemme check that. |
I don't think it updates the original CSV file when you use the GUI |
Yeah it doesn't that was a dumb comparison. My b. Either way, it seems to understand adding in the pre-req in the GUI no problem. |
I don't think this happens often enough so we could probably manually fix it when it happens with the GUI |
Sounds good to me. |
Out of curiosity, I tried it out in the Julia package... and it works fine. INTL101 or 102 1 correctly had CAT 1 as a pre-req. Admittedly, those degree plans exist abstractly; I don't recall how to link them to a curriculum to accurately reproduce our situation. Just a curious fact then I guess. |
In IS28/Sixth,
This seems to occur for other degree plans in the major
"INTL 101 or 102" gets parsed as INTL 101; both INTL 101 and 102 require at least one quarter of a writing sequence.
The text was updated successfully, but these errors were encountered: