Skip to content

Commit 5ea36f9

Browse files
author
Suhas Hariharan
committed
removes COVID grade freeze case which is causing other issues with cumulative GPA calculation
Signed-off-by: Suhas Hariharan <[email protected]>
1 parent 399c3fb commit 5ea36f9

File tree

1 file changed

+17
-11
lines changed

1 file changed

+17
-11
lines changed

src/js/components/CumulativeGPA.vue

Lines changed: 17 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,7 @@ export default {
138138
this.currently = CURRENTLY.ERROR;
139139
}
140140
},
141-
calculate_cumulative_gpa (current_courses, current_term, current_semester) {
141+
calculate_cumulative_gpa (current_courses, current_term, is_second_semester) {
142142
const list_of_gpas = [];
143143
const all_courses = [];
144144
const credit_hour_list = [];
@@ -198,27 +198,33 @@ export default {
198198
}
199199
// Calculates cumulative GPA based on credit hours per semester and gpa for each semester.
200200
const cumulative_gpa = Promise.all(fetches).then(function () {
201-
let include_current_semester = false;
202-
if (current_courses.length !== 0) {
203-
for (let i = 0; i < current_courses.length; i++) {
204-
if (new URL(current_courses[i].link).searchParams.get("begdate")) {
205-
include_current_semester = true;
206-
}
207-
}
201+
let include_current_semester;
202+
let current_semester;
203+
if (second_semester) {
204+
current_semester = "S2";
205+
}
206+
else {
207+
current_semester = "S1";
208+
}
209+
if (current_term == current_semester) {
210+
include_current_semester = false;
211+
}
212+
else {
213+
include_current_semester = true;
208214
}
209215
// Handles edge case where grade history page is updated before semester end, removes the old value and counts the latest version of the current semester.
210-
// Temporarily removing this as this edge case is causing double counting current semester, this can be re-implemented later, but is likely unimportant as the previous instance of this being needed was due to COVID-19 grade freeze.
211-
/*
216+
// Removed as it seems to be causing other issues with calculating cumulative GPA, this can be reworked if necessary, but it is a rare case as it only occured when SAS implemented a grade freeze due to COVID resulting in grade history being updated prematurely.
217+
/* temporarily
212218
if (current_term_history === current_term && include_current_semester && current_term_grades.length === 2 && current_semester) {
213219
all_courses.splice(all_courses.indexOf(current_term_grades[1]), 1);
214220
} else if (current_term_history === current_term && include_current_semester && current_term_grades.length === 1 && current_semester === false) {
215221
all_courses.splice(all_courses.indexOf(current_term_grades[0]), 1);
216222
}
223+
*/
217224
218225
if (include_current_semester) {
219226
all_courses.push(current_courses);
220227
}
221-
*/
222228
let total_count = 0;
223229
let total_gpa = 0;
224230
for (let i = 0; i < all_courses.length; i++) {

0 commit comments

Comments
 (0)