Feat/why projects fits you section. - #935
Merged
komalharshita merged 14 commits intoJul 12, 2026
Merged
Conversation
added 4 commits
June 8, 2026 16:47
- utils/recommender.py: normalize raw score to 0-10 scale and attach as match_score field on each returned project dict - static/script.js: render match score badge (label + number + bar) inside buildProjectCard() between title and description - static/style.css: add styles for .project-match-score, .score-label, .score-value, .score-bar, .score-bar-fill Resolves issue: show score/10 on recommended projects for clarity
|
Someone is attempting to deploy a commit to the komalsony234-1530's projects Team on Vercel. A member of the Team first needs to authorize it. |
Author
|
Hi maintainers, |
Owner
|
@Soumya1Kesharwani kindly resolve the merge conflicts |
added 7 commits
July 6, 2026 21:49
…utton HTML, and missing variable declarations
# Conflicts: # data/projects.json
Soumya1Kesharwani
force-pushed
the
feat/why-fits-you-only
branch
from
July 6, 2026 18:30
4baf315 to
c704437
Compare
Author
|
Ma'am please review the changes and if everything looks good kindly merge it to main. |
Author
|
Ma'am I have resolved merge conflicts.. please merge it to main code.
…On Sun, Jul 5, 2026, 22:02 Komal Harshita ***@***.***> wrote:
Reopened #935 <#935>.
—
Reply to this email directly, view it on GitHub
<#935?email_source=notifications&email_token=BW5UIZC5JLJWWQOBTS6URCD5DJ7IJA5CNFSNUABQM5UWIORPF5TWS5BNNB2WEL2JONZXKZKFOZSW45CON52GSZTJMNQXI2LPNYXTENZVG43DONBTGIYDFJTSMVQXG33OU5WWK3TUNFXW5JLFOZSW45FMMZXW65DFOJPWG3DJMNVQ#event-27576743202>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BW5UIZFEOKGC7FFH2A4N7CL5DJ7IJAVCNFSNUABGKJSXA33TNF2G64TZHMYTCOJWGI2DAOJYGQ5US43TOVSTWNBWHA4DQMRUGE4DRILWAI>
.
Triage notifications, keep track of coding agent tasks and review pull
requests on the go with GitHub Mobile for iOS
<https://github.com/notifications/mobile/ios/BW5UIZBIW4F5Y4WLYTHDFYL5DJ7IJA5CNFSNUABQM5UWIORPF5TWS5BNNB2WEL2JONZXKZKFOZSW45CON52GSZTJMNQXI2LPNYXTENZVG43DONBTGIYDFJTSMVQXG33OU5WWK3TUNFXW5JLFOZSW45FKMZXW65DFOJPWS33T>
and Android
<https://github.com/notifications/mobile/android/BW5UIZERPGWSOTAUNOUB2MD5DJ7IJA5CNFSNUABQM5UWIORPF5TWS5BNNB2WEL2JONZXKZKFOZSW45CON52GSZTJMNQXI2LPNYXTENZVG43DONBTGIYDFJTSMVQXG33OU5WWK3TUNFXW5JLFOZSW45FOMZXW65DFOJPWC3TEOJXWSZA>.
Download it today!
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Author
|
Ma'am please check my pr.
On Tue, Jul 7, 2026, 22:58 Soumya Kesharwani ***@***.***>
wrote:
… Ma'am I have resolved merge conflicts.. please merge it to main code.
On Sun, Jul 5, 2026, 22:02 Komal Harshita ***@***.***>
wrote:
> Reopened #935 <#935>.
>
> —
> Reply to this email directly, view it on GitHub
> <#935?email_source=notifications&email_token=BW5UIZC5JLJWWQOBTS6URCD5DJ7IJA5CNFSNUABQM5UWIORPF5TWS5BNNB2WEL2JONZXKZKFOZSW45CON52GSZTJMNQXI2LPNYXTENZVG43DONBTGIYDFJTSMVQXG33OU5WWK3TUNFXW5JLFOZSW45FMMZXW65DFOJPWG3DJMNVQ#event-27576743202>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/BW5UIZFEOKGC7FFH2A4N7CL5DJ7IJAVCNFSNUABGKJSXA33TNF2G64TZHMYTCOJWGI2DAOJYGQ5US43TOVSTWNBWHA4DQMRUGE4DRILWAI>
> .
> Triage notifications, keep track of coding agent tasks and review pull
> requests on the go with GitHub Mobile for iOS
> <https://github.com/notifications/mobile/ios/BW5UIZBIW4F5Y4WLYTHDFYL5DJ7IJA5CNFSNUABQM5UWIORPF5TWS5BNNB2WEL2JONZXKZKFOZSW45CON52GSZTJMNQXI2LPNYXTENZVG43DONBTGIYDFJTSMVQXG33OU5WWK3TUNFXW5JLFOZSW45FKMZXW65DFOJPWS33T>
> and Android
> <https://github.com/notifications/mobile/android/BW5UIZERPGWSOTAUNOUB2MD5DJ7IJA5CNFSNUABQM5UWIORPF5TWS5BNNB2WEL2JONZXKZKFOZSW45CON52GSZTJMNQXI2LPNYXTENZVG43DONBTGIYDFJTSMVQXG33OU5WWK3TUNFXW5JLFOZSW45FOMZXW65DFOJPWC3TEOJXWSZA>.
> Download it today!
> You are receiving this because you were mentioned.Message ID:
> ***@***.***>
>
|
Owner
|
Ok, approved for merge! |
komalharshita
approved these changes
Jul 12, 2026
Author
|
Thanks Ma'am
…On Sun, Jul 12, 2026, 16:38 Komal Harshita ***@***.***> wrote:
Merged #935 <#935> into main.
—
Reply to this email directly, view it on GitHub
<#935?email_source=notifications&email_token=BW5UIZGCAOPIIMQZRAH74B35ENWS7A5CNFSNUABQM5UWIORPF5TWS5BNNB2WEL2JONZXKZKFOZSW45CON52GSZTJMNQXI2LPNYXTENZYG4ZDANJVGU3TLJTSMVQXG33OU5WWK3TUNFXW5JLFOZSW45FMMZXW65DFOJPWG3DJMNVQ#event-27872055575>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BW5UIZA5MD45FPTIL7U2XI35ENWS7AVCNFSNUABGKJSXA33TNF2G64TZHMYTCOJWGI2DAOJYGQ5US43TOVSTWNBWHA4DQMRUGE4DRILWAI>
.
Triage notifications, keep track of coding agent tasks and review pull
requests on the go with GitHub Mobile for iOS
<https://github.com/notifications/mobile/ios/BW5UIZFH4FB6IWT2EKIK2XD5ENWS7A5CNFSNUABQM5UWIORPF5TWS5BNNB2WEL2JONZXKZKFOZSW45CON52GSZTJMNQXI2LPNYXTENZYG4ZDANJVGU3TLJTSMVQXG33OU5WWK3TUNFXW5JLFOZSW45FKMZXW65DFOJPWS33T>
and Android
<https://github.com/notifications/mobile/android/BW5UIZEM65VS6DZX7ZHVLBL5ENWS7A5CNFSNUABQM5UWIORPF5TWS5BNNB2WEL2JONZXKZKFOZSW45CON52GSZTJMNQXI2LPNYXTENZYG4ZDANJVGU3TLJTSMVQXG33OU5WWK3TUNFXW5JLFOZSW45FOMZXW65DFOJPWC3TEOJXWSZA>.
Download it today!
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #129
Hi @komalharshita Ma'am -
This PR introduces a new feature that provides users with a personalized, dynamically generated explanation of exactly why a specific project was recommended to them (e.g., "This project utilizes your skills in Python, fits your current experience level, and aligns closely with your interests"). It also includes a critical backend fix to data_loader.py.
Changes Included in this PR:
Backend Recommendation Logic (utils/recommender.py)
i. Updated score_single_project() to accurately track and return exactly which parameters matched the user's input via a new match_details dictionary.
ii.Updated get_recommendations() to dynamically generate a natural language match_explanation sentence tailored to the user's profile.
Frontend Details Page (templates/project.html)
for seamless client-side injection.i. Added an empty
ii. Refactored the way variables are passed to JavaScript by adopting a secure tag approach to prevent template rendering conflicts.
JavaScript UI Rendering (static/script.js)
i. Updated the recommendation grid generation to securely encode and pass the match_explanation via a URL parameter (?reason=...).
ii. Implemented a new loadExplanation() function that automatically reads the URL parameter on the project detail page and renders the explanation.
Data Loader Stability Fix (utils/data_loader.py)
i. Added _cache_lock to make project loading fully thread-safe.
ii. Added extensive try/except blocks to gracefully catch FileNotFoundError and json.JSONDecodeError to prevent application crashes when the projects.json file is malformed or missing.
After updating--
I tested the feature locally and the match score functionality is working fine.Please let me know if there are any edge cases I should check.
Kindly review the changes. If everything looks good, please merge this PR into main.
Thanks for your time.