Skip to content

Commit

Permalink
Redo A&S 100 and 120 Credit Minimum Requirements (#686)
Browse files Browse the repository at this point in the history
* disallow transfer credit for a&s 100 credits

* add 120 total credits requirement for A&S

* revert name change for A&S Credit requirement & update tests

* use shared 120 credit minimum with other colleges

* clean up 120 credit requirement

* make AS1 and AS2 use the same requirement descriptions

* minor rewording of description
  • Loading branch information
noschiff authored Apr 25, 2022
1 parent c04efa9 commit 680bc24
Show file tree
Hide file tree
Showing 4 changed files with 136 additions and 137 deletions.
12 changes: 8 additions & 4 deletions src/requirements/data/colleges/asFA2020.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,17 +8,21 @@ const casFA2020Requirements: readonly CollegeOrMajorRequirement[] = [
'100 credits in Arts & Sciences are required. ' +
'Students can take more than 20 credits outside of the College as long as they take 100 credits within; ' +
'they can also take all their credits in Arts & Sciences and accumulate more than 120. ' +
'Note: AP, IB, and A-Level credits count toward the 120 total credits but not toward the 100 A&S credits.',
'Note: AP, IB, and A-Level credits do not count toward this requirement.',
source: 'https://as.cornell.edu/education/degree-requirements',
checker: [
(course: Course): boolean =>
course.acadGroup.includes('AS') ||
course.subject === 'CS' ||
(course.catalogDistr?.includes('-AS') ?? false),
// Note that this does not completely agree with the 120 total academic credits because that incorrectly includes courses cross-listed with PE
course.subject !== 'PE' &&
!ifCodeMatch(course.catalogNbr, '10**') &&
(course.acadGroup.includes('AS') ||
course.subject === 'CS' ||
(course.catalogDistr?.includes('-AS') ?? false)),
],
fulfilledBy: 'credits',
perSlotMinCount: [100],
allowCourseDoubleCounting: true,
disallowTransferCredit: true,
},
{
name: 'First-Year Writing Seminars (FWS)',
Expand Down
2 changes: 1 addition & 1 deletion src/requirements/data/colleges/asPreFA2020.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ const casPreFA2020Requirements: readonly CollegeOrMajorRequirement[] = [
'100 credits in Arts & Sciences are required. ' +
'Students can take more than 20 credits outside of the College as long as they take 100 credits within; ' +
'they can also take all their credits in Arts & Sciences and accumulate more than 120. ' +
'Note: AP, IB, and A-Level credits count toward the 120 total credits but not toward the 100 A&S credits.',
'Note: AP, IB, and A-Level credits do not count toward this requirement.',
source: 'https://as.cornell.edu/education/old-degree-requirements',
checker: [
(course: Course): boolean =>
Expand Down
Loading

0 comments on commit 680bc24

Please sign in to comment.