-
Notifications
You must be signed in to change notification settings - Fork 831
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: add support for dimensions parameter to OpenAIEmbedding #2215
chore: add support for dimensions parameter to OpenAIEmbedding #2215
Conversation
Acrolinx ScorecardsA minimum total score of 80 is required. Select the total score link to review all feedback on clarity, consistency, tone, brand, terms, spelling, grammar, readability, and inclusive language. You should fix all spelling errors regardless of your total score. Fixing spelling errors helps maintain customer trust in overall content quality.
More information about Acrolinx |
/azp run |
Acrolinx ScorecardsA minimum total score of 80 is required. Select the total score link to review all feedback on clarity, consistency, tone, brand, terms, spelling, grammar, readability, and inclusive language. You should fix all spelling errors regardless of your total score. Fixing spelling errors helps maintain customer trust in overall content quality.
More information about Acrolinx |
Azure Pipelines successfully started running 1 pipeline(s). |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #2215 +/- ##
==========================================
- Coverage 86.19% 86.17% -0.03%
==========================================
Files 327 327
Lines 16705 16715 +10
Branches 1501 1495 -6
==========================================
+ Hits 14399 14404 +5
- Misses 2306 2311 +5 ☔ View full report in Codecov by Sentry. |
Acrolinx ScorecardsA minimum total score of 80 is required. Select the total score link to review all feedback on clarity, consistency, tone, brand, terms, spelling, grammar, readability, and inclusive language. You should fix all spelling errors regardless of your total score. Fixing spelling errors helps maintain customer trust in overall content quality.
More information about Acrolinx |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Acrolinx ScorecardsA minimum total score of 80 is required. Select the total score link to review all feedback on clarity, consistency, tone, brand, terms, spelling, grammar, readability, and inclusive language. You should fix all spelling errors regardless of your total score. Fixing spelling errors helps maintain customer trust in overall content quality.
More information about Acrolinx |
Related Issues/PRs
Closes #2177
What changes are proposed in this pull request?
Adding support for
dimensions
parameter when calling OpenAI Embeddings endpoint to reduce the size of the embedding vecor.How is this patch tested?
Added
Extra Param Usage
test to OpenAIEmbeddingSuiteDoes this PR change any dependencies?
Does this PR add a new feature? If so, have you added samples on website?
Added example to existing OpenAI.md
website/docs/documentation
folder.Make sure you choose the correct class
estimators/transformers
and namespace.DocTable
points to correct API link.yarn run start
to make sure the website renders correctly.<!--pytest-codeblocks:cont-->
before each python code blocks to enable auto-tests for python samples.WebsiteSamplesTests
job pass in the pipeline.