Skip to content
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

proposed docs #237

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

proposed docs #237

wants to merge 6 commits into from

Conversation

mkozakov
Copy link
Contributor

@mkozakov mkozakov commented Nov 4, 2024

This pull request introduces several changes to the cohere-openapi.yaml file, primarily focusing on refining the descriptions and parameters for the ReRank API. The updates aim to provide clearer guidance and improve the user experience when interacting with the API.

  • Description Updates:
    • The description field for the model identifier has been enhanced to include a list of supported models (rerank-english-v3.0, rerank-multilingual-v3.0, rerank-english-v2.0, and rerank-multilingual-v2.0).
    • The query description now specifies that the search query is used to rerank documents, with a maximum length of 500 tokens. It also provides a link to a tokenization guide for reference.
    • The documents description has been modified to clarify that it accepts a list of strings for reranking against the search query. It mentions automatic truncation for long documents and recommends a limit of 1,000 documents for optimal performance.
  • Parameter Adjustments:
    • The max_chunks_per_doc parameter has been replaced with max_document_length, which truncates documents exceeding the specified length. The default value has been increased from 10 to 4096.
    • The rank_fields, return_documents, and max_chunks_per_doc parameters have been removed.

Copy link

github-actions bot commented Nov 4, 2024

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.

1 participant