project recommendations using Groq#668
Conversation
|
@Userunknown84 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. |
|
Hi, please tell me if this is okay, as I'm trying with the API this time.
Please tell me if it's accepted or rejected.
…On Thu, 28 May, 2026, 2:54 pm vercel[bot], ***@***.***> wrote:
*vercel[bot]* left a comment (komalharshita/DevPath#668)
<#668 (comment)>
@Userunknown84 <https://github.com/Userunknown84> is attempting to deploy
a commit to the *komalsony234-1530's projects* Team on Vercel
<https://vercel.com>.
A member of the Team first needs to authorize it
<https://vercel.com/git/authorize?team=komalsony234-1530's%20projects&slug=komalsony234-1530s-projects&teamId=team_4uoKBbp41VaIZKo2f2nQAKNx&type=github&job=%7B%22headInfo%22%3A%7B%22sha%22%3A%227a1d75698dc8e91049d038974d6d8d3d051c4a01%22%7D%2C%22id%22%3A%22QmcsqnrRtmYQoFKwcD8h8xRdAAVkraVuL4jdHxGFHxqWxd%22%2C%22org%22%3A%22komalharshita%22%2C%22prId%22%3A668%2C%22repo%22%3A%22DevPath%22%7D>
.
—
Reply to this email directly, view it on GitHub
<#668?email_source=notifications&email_token=BC5FCOIS7ORVWTIR3RXVK4345AAT7A5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTINJWGI2TINZZGQ42M4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJLDGN5XXIZLSL5RWY2LDNM#issuecomment-4562547949>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BC5FCOMR3U3O5MPAODVGMPT45AAT7AVCNFSM6AAAAACZQSTBD6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHM2DKNRSGU2DOOJUHE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
|
Thank you for the contribution. I reviewed the changes and appreciate the effort to explore AI-generated project recommendations. However, this PR fundamentally replaces DevPath's existing recommendation engine with an external Groq-based workflow. This introduces a mandatory external dependency, requires API key configuration, removes existing validation logic, changes core application behavior, and adds significant maintenance and deployment complexity. The implementation also assumes AI responses will always return valid JSON, exposes exception details in API responses, introduces additional frontend changes unrelated to the primary feature, and does not include automated tests to validate the new behavior. Because this proposal changes the architecture of the project rather than addressing a specific issue within the current system, it is not suitable for merging in its current form. For these reasons, I’m closing this pull request. Thank you for contributing to DevPath. |
Summary [required]
->I use the groq api key to show projects simultaneously and no stored project , fetch directly from api.
->Project , code everything fetch thorugh api and no stored project .
-> For this i used a model "llama" to show projects when user enter skill then project show thorugh this model.
Related Issue [required]
Closes #
Type of Change [required]
data/projects.jsonWhat Was Changed [required]
utils/recommender.pyclear_cache()functiontests/test_basic.pyHow to Test This PR [required]
git checkout your-branch-namepip install -r requirements.txtpython app.pypython tests/test_basic.pyExpected test output:
Test Results [required]
Screenshots (if UI change)
Self-Review Checklist [required]
feat/,fix/,docs/,data/,style/,test/python tests/test_basic.pyand all 27 tests passflake8 .locally and there are no errorsprint()orconsole.log()debug statementsNotes for Reviewer