Skip to content

199 similarity list#204

Merged
LC0229 merged 10 commits intomainfrom
199-SimilarityList
Feb 16, 2026
Merged

199 similarity list#204
LC0229 merged 10 commits intomainfrom
199-SimilarityList

Conversation

@cthotti
Copy link
Collaborator

@cthotti cthotti commented Feb 5, 2026

Summary

Describe the purpose of this pull request. What does it do, and why is it needed?

  • This feature is for users to find similar characteristics between the alumni and themselves.
  • I also changed the layout and design of student and alumni profiles, plus connect page

Closes #(issue number) if applicable.
#199

Changes Made

  • changed groq model to llama
  • updated the calls to and from groq
  • created new validators to check responses
  • ensured that both student and alumni info are sent to backend

Briefly summarize the major changes:


Testing

How was this change tested?

  • Manual end-to-end testing
  • Unit Tests
  • CI checks passed

@vercel
Copy link

vercel bot commented Feb 5, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
opportune Ready Ready Preview, Comment Feb 15, 2026 2:40am

Copy link
Collaborator

@n1sh1thaS n1sh1thaS left a comment

Choose a reason for hiding this comment

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

Could you remove the new design changes into a different branch and only keep the similarity list feature in this PR. Thanks!

Copy link
Collaborator

@n1sh1thaS n1sh1thaS left a comment

Choose a reason for hiding this comment

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

Could you also remove personal details (like name) from the data that is passed into Groq. You can just have the output refer to the student as "you" and the alumni as "this alumni" or something along those lines.

@LC0229 LC0229 merged commit 6ff665f into main Feb 16, 2026
5 checks passed
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.

4 participants