Skip to content

Conversation

@RiceViz
Copy link
Contributor

@RiceViz RiceViz commented Aug 21, 2025

Added All Of The Required Functions For This Assignment In 'lesson_07.ts' And 'part_d.ts' While Passed All Test Cases For Both

Lessons Learned:
Gets a refresher on Binary Search and Fibonacci Sequence and using dynamic programming to solve it

Problems Faced:
I faced no real problems for this assignment

Copy link
Contributor

@anthonydmays anthonydmays left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎓 Automated Grading Report

Student: Tyran Rice
Date: 8/21/2025

Feedback

Tyran Rice has successfully met all functional requirements for the assignment. The functions in both lesson7.ts and part_d.ts have been implemented correctly. The compareStrings function correctly returns -1, 0, or 1 based on the lexicographic distance. The computeFactorial function handles edge cases and calculates factorials accurately. The getFirstNFibonacciNumbers function generates the correct Fibonacci sequence. The binarySearch function is implemented recursively and correctly identifies the index of the target value. In part_d.ts, the isWithinRange function correctly checks if a number is within a specified range, and isValidTriangle checks for a valid right triangle. The getSeason function accurately maps months to seasons. Technically, the code is well-structured, follows TypeScript conventions, and all GitHub checks have passed, indicating no syntax or type errors. No stretch criteria were provided or attempted, so the stretch score is 0. Overall, excellent work on meeting the assignment requirements with clean and efficient code.


This is an automated preliminary review. Please review and adjust before finalizing.

@RiceViz RiceViz deleted the feat/lesson_07/conditionals branch August 21, 2025 17:37
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 this pull request may close these issues.

2 participants