Skip to content

Assignment 2#113

Closed
aakashpatel379 wants to merge 2 commits intoUofT-DSI:mainfrom
aakashpatel379:assignment-2
Closed

Assignment 2#113
aakashpatel379 wants to merge 2 commits intoUofT-DSI:mainfrom
aakashpatel379:assignment-2

Conversation

@aakashpatel379
Copy link

@aakashpatel379 aakashpatel379 commented Mar 5, 2026

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)

Assignment 2 Project

What did you learn from the changes you have made?

Learnt how to implement AI system with couple of services catering to different needs

Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?

No. Made into expansive project so as to learn working with Semantic, Function calling, web search etc

Were there any challenges? If so, what issue(s) did you face? How did you overcome it?

Challenges in getting the routing correct and trying to handle different possibilities of prompt. Added logic to identify Prompt intent by looking for presence of certain keywords, which then guide further sequence of operations

How were these changes tested?

Tested mostly with the help of locally spawned servers and attempting different manual test prompts to validate different outcomes for correctness.

A reference to a related issue in your repository (if applicable)

Checklist

  • I can confirm that my changes are working as intended

@github-actions
Copy link

github-actions bot commented Mar 5, 2026

Hello, thank you for your contribution. If you are a participant, please close this pull request and open it in your own forked repository instead of here. Please read the instructions on your onboarding Assignment Submission Guide more carefully. If you are not a participant, please give us up to 72 hours to review your PR. Alternatively, you can reach out to us directly to expedite the review process.

@aakashpatel379
Copy link
Author

Requesting @vishnouvina / @Dmytro-Bonislavskyi for review. Thanks!

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.

1 participant