Skip to content
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

## TH27 - Moved to keep #30 clean #35

Closed
ArturoAmaya opened this issue Jun 15, 2022 · 3 comments · Fixed by #43
Closed

## TH27 - Moved to keep #30 clean #35

ArturoAmaya opened this issue Jun 15, 2022 · 3 comments · Fixed by #43

Comments

@ArturoAmaya
Copy link
Owner

TH27

Name: uploaded uses "Theater," generated uses "Theatre" (which I think is more correct)

Complexity is 2 less in the generated version

Only uploaded has prerequisites from TDTR 10 -> TDDM 101 and TDHD XXX. Only generated has prerequisites from TDDM 1 to 101. The main difference here I think is that for TDDM 101's prerequisites, the uploaded version says it's TDTR 10 while the generated version says it's TDDM 1. According to the course catalog, it's only TDDM 1.

image

Another difference is that the uploaded version uses "UD Elective Mvmnt n," while the generated version calls them all "UD Elective Movement." Similarly for Core Movement. Maybe we might want to consider adding numbers to duplicate course names? (This is the case for degree plans too, eg "GE") ⚠️

Revelle

The generated version has one more curr. compl. Core movements have been moved from terms 9 and 10 in the uploaded version to terms 2 and 4 in the generated version (This is the case for many other degree plans)

The uploaded version calls it "CHEM GE" while the generated version calls it "CHEMISTRY GE (see note)"

Muir

Uploaded: "MCWP 40"; generated: "MCWP 40/GE"

Marshall

Uploaded: "DOC 1"; generated: "DOC 1/DEI"

Warren

Interestingly, uploaded: "PHIL/POLI 1"; generated: "PHIL/POLI 27." I think 1/2 here means "PHIL/POLI" is just a generic elective sequence, while the generated version has specific course

Sixth

Uploaded: "CAT 125"; generated: "PRACTICUM OR CAT 125"

Originally posted by @SheepTester in #30 (comment)

@ArturoAmaya
Copy link
Owner Author

ArturoAmaya commented Jun 15, 2022

  • Complexity is 2 less in the generated version because of the following changes:
    • +1 complexity for TDDM1 from being a prereq for TDDM101, which as you said is correct.
    • -2 complexity for TDTR10 not being a prereq of TDDM101 or TDHD XXX. TDTR10 isn't a prereq of TDDM101, as you pointed out.
    • -1 complexity for TDHDXXX not being unblocked by TDTR10.
      I checked the Theatre and Dance catalog and there are only 4 TDHD courses. They all require TDTR10, so we should add that in.

The generated names being a bit more descriptive is fine to me, and more in line with the online plan generator.

@ArturoAmaya
Copy link
Owner Author

About numbering the electives, I think that's a fairly good idea. Would courses like MCWP40/GE factor into the the numbering?

@SheepTester
Copy link
Collaborator

I think we'd probably just chop off the /GE /DEI /AWP stuff like you mentioned in "Issues"

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

Successfully merging a pull request may close this issue.

2 participants