Lauren Steenkamp

[Lauren-Robyn]

No description provided.

[Copilot]

Pull Request Overview

This PR implements several previously stubbed-out functions in student_code.py, including mathematical calculations, grade evaluation, performance analysis, and number filtering utilities.

Key Changes:

• Implementation of mathematical functions (sum_of_squares, even_numbers, odd_numbers)
• Implementation of grade/performance evaluation functions (evaluate_performance, calculate_cumulative_performance, analyze_improvement)
• Addition of debug print statements at module level

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The function returns the result of print(total) which is None, instead of returning the actual total value. Remove print() and return total directly to match the documented return type of int.

Suggested change

	return print(total)
	return total

[Copilot]

Module-level print statement should be removed. This creates side effects when the module is imported and is not part of the function's API.

Suggested change

	print(sum_of_squares(100))
	if __name__ == "__main__":
	print(sum_of_squares(100))

[Copilot]

The function parameters grades and min_pass are being overwritten with hardcoded values, making the function ignore its inputs. Remove these assignments and use the parameters directly.

Suggested change

	grades =[80, 70, 90]
	min_grade_passing = 75
	total = 0
	for num in grades:
	total += num
	average = total // len(grades)
	if average < min_grade_passing:
	total = 0
	for num in grades:
	total += num
	average = total // len(grades)
	if average < min_pass:

[Copilot]

Using integer division (//) for calculating average will truncate decimal values. Use regular division (/) to get accurate averages for grade comparison.

Suggested change

	average = total // len(grades)
	average = total / len(grades)

[Copilot]

The function prints the result instead of returning it. Replace print() calls with return statements to match the documented return type of str. Also, the hardcoded variable min_grade_passing should be replaced with the min_pass parameter.

Suggested change

	grades =[80, 70, 90]
	min_grade_passing = 75
	total = 0
	for num in grades:
	total += num
	average = total // len(grades)
	if average < min_grade_passing:
	print("Fail")
	else:
	print("Pass")
	# grades =[80, 70, 90] # Removed hardcoded grades
	# min_grade_passing = 75 # Removed hardcoded min_grade_passing
	total = 0
	for num in grades:
	total += num
	average = total // len(grades)
	if average < min_pass:
	return "Fail"
	else:
	return "Pass"

[Copilot]

The range should be range(1, n + 1) to include n itself, as the docstring states 'from 1 to n' (inclusive).

Suggested change

	for i in range(1, n):
	for i in range(1, n + 1):

[Copilot]

Module-level print statement should be removed. This creates side effects when the module is imported and is not part of the function's API.

Suggested change

	print(even_numbers(20))
	if __name__ == "__main__":
	print(even_numbers(20))

[Copilot]

The range should be range(1, n + 1) to include n itself, as the docstring states 'from 1 to n' (inclusive).

Suggested change

	for i in range(1,n):
	for i in range(1, n + 1):

[Copilot]

The return statement is inside the loop, causing the function to return after finding the first odd number. Unindent this line to be at the same level as the for loop.

Suggested change

	return odd_numbers
	return odd_numbers

[Copilot]

Module-level print statement should be removed. This creates side effects when the module is imported and is not part of the function's API.

Suggested change

print(odd_numbers(20))