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

Updating readme code #75

Merged
merged 1 commit into from
Jan 8, 2025
Merged

Updating readme code #75

merged 1 commit into from
Jan 8, 2025

Conversation

ankush-cohere
Copy link
Contributor

@ankush-cohere ankush-cohere commented Jan 8, 2025

This pull request introduces changes to the README.md file, focusing on updating the configuration and usage of the Cohere Compass API.

Changes:

  • API Configuration: The url variable has been replaced with api_url and parser_url, providing a more structured approach to handling API endpoints.
  • Authentication: A new variable, bearer_token, has been added to support authentication with the API. This token is passed to the CompassClient when creating the index.
  • Parsing Options: The process_folder method now includes the recursive parameter, enabling recursive folder processing during parsing.
  • Search Results: The search results retrieval has been modified to use the result variable, ensuring consistent access to the search outcomes.

@ankush-cohere ankush-cohere requested a review from a team as a code owner January 8, 2025 09:50
@ankush-cohere ankush-cohere force-pushed the update/readme_code branch 2 times, most recently from f58731c to 7dea1b7 Compare January 8, 2025 10:04
Copy link

@andytran24 andytran24 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks better

@ankush-cohere ankush-cohere merged commit 0db8e2d into main Jan 8, 2025
11 checks passed
@ankush-cohere ankush-cohere deleted the update/readme_code branch January 8, 2025 15:09
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.

3 participants