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

SOCI 60 is a prereq of SOCI UD Methodology #34

Closed
ArturoAmaya opened this issue Jun 15, 2022 · 2 comments · Fixed by #50
Closed

SOCI 60 is a prereq of SOCI UD Methodology #34

ArturoAmaya opened this issue Jun 15, 2022 · 2 comments · Fixed by #50

Comments

@ArturoAmaya
Copy link
Owner

SO31

Uploaded, generated

The uploaded version uses "SOCI 10-50," which IMO is more confusing than the generated version's "SOCI 10, 20, 30, 40, OR 50" because it suggests all of them need to be taken, when only one needs to be taken.

It seems that generally, the uploaded names are more abbreviated, which is good because the names in the generated version tend to be clipped off because they're too long:
image
The uploaded version uses a single letter for "concentration," "methodology," or "elective."

Only the uploaded version has a prereq from SOCI 60 to "SOCI UD M." This is not obvious from the given data we have but I think it might be important considering that SOCI 60 is a prereq for many upper div sociology courses ⚠️

Revelle

The generated version has "(DEI APPROVED)" and "(see note)"

Uploaded: "FINE ARTS GE"; generated: "FINE ARTS"

Uploaded: "AHI/Elective"; generated: "AHI or ELECTIVE"

The uploaded version also has a prereq from HUM 2 to HUM 3, while the generated version doesn't. This is a result of #25 because HUM 1 is enough for HUM 3

Originally posted by @SheepTester in #30 (comment)

@ArturoAmaya
Copy link
Owner Author

ArturoAmaya commented Jun 15, 2022

In order

  • Agreed that 10-50 is more confusing, it also implies that a course like SOCI 11 exists (it doesn't) and that it would satisfy the requirement.
  • Shortening the names might be worth doing. I know that the web tool lets you write annotations into the courses, so we could write an abbreviated name and the full version in the annotation if we can figure out how to annotate from the .csv
  • I checked at the SOCI department website and they say that SOCI 60 is specifically a pre-req for every UD Methods course, so we should definitely add that in as a pre-req for "SOCI UD Methodology". We can probably add it to the pre-reqs document and re-download that, so that all our info is consistent.
  • What do you think of removing the "see note"? We don't have any note for people to see 😵‍💫 . As for the other small name differences, that's no big deal right now, methinks.

@SheepTester
Copy link
Collaborator

  • What do you think of removing the "see note"? We don't have any note for people to see 😵‍💫 . As for the other small name differences, that's no big deal right now, methinks.

Yeah I can go through and see what parentheses and slashes there are (some we might want to keep if they instead list alternative courses)

@SheepTester SheepTester changed the title ## SO31 - Moved to keep #30 clean SOCI 60 is a prereq of SOCI UD Methodology Jun 17, 2022
SheepTester added a commit to SheepTester-forks/ExploratoryCurricularAnalytics that referenced this issue Jun 21, 2022
SOCI 60 -> SOCI- UD METHODOLOGY (ArturoAmaya#34)
TDTR 10 -> TDHD XXX (ArturoAmaya#48)
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