Fixed input arg count issue with get_embedding_request_data utility function - Githubissues

pinecone-io / sample-apps

15 stars 7 forks source link

Fixed input arg count issue with get_embedding_request_data utility function #9

Closed jackyliang closed 1 month ago

jackyliang commented 1 month ago

Problem

Fixed an issue where I omitted self into input arg of the get_embedding_request_data utility function

Solution

Encountered this error after deploying to Vercel in the add-shop-the-look branch

Fixed it by adding self back into the input args of the offending utility function

Type of Change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] Infrastructure change (CI configs, etc)
[ ] Non-code change (docs, etc)
[ ] None of the above: (explain here)

Test Plan

I tested on localhost and it fixes the problem. Will double check once deployed to Vercel.