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

feat: Make docs content be splited by users #303

Merged
merged 1 commit into from
May 1, 2024
Merged

Conversation

whiterabbit1983
Copy link
Contributor

@whiterabbit1983 whiterabbit1983 commented Apr 30, 2024

🚀 This description was created by Ellipsis for commit ceb5f53

Summary:

Enhanced document handling and embedding functionalities, and refactored agent instructions for improved flexibility.

Key points:

  • Enhanced document handling and embedding functionalities
  • Updated CreateDoc model to accept both string and list of strings for content
  • Refactored embedding functionalities to include embedding_model_id
  • Adjusted environment configurations to support new embedding model ID
  • Changed agent instructions handling from string to list of strings for flexibility

Generated with ❤️ by ellipsis.dev

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

👍 Looks good to me!

  • Reviewed the entire pull request up to 630785b
  • Looked at 926 lines of code in 7 files
  • Took 1 minute and 7 seconds to review
More info
  • Skipped 0 files when reviewing.
  • Skipped posting 0 additional comments because they didn't meet confidence threshold of 50%.

Workflow ID: wflow_TzhdUtYNRCFbofxp


You can customize Ellipsis with review rules, user-specific overrides, quiet mode, and more. See docs.

agents-api/agents_api/autogen/openapi_model.py Outdated Show resolved Hide resolved
agents-api/agents_api/autogen/openapi_model.py Outdated Show resolved Hide resolved
agents-api/agents_api/autogen/openapi_model.py Outdated Show resolved Hide resolved
agents-api/agents_api/autogen/openapi_model.py Outdated Show resolved Hide resolved
agents-api/agents_api/autogen/openapi_model.py Outdated Show resolved Hide resolved
agents-api/agents_api/autogen/openapi_model.py Outdated Show resolved Hide resolved
agents-api/agents_api/autogen/openapi_model.py Outdated Show resolved Hide resolved
agents-api/agents_api/autogen/openapi_model.py Outdated Show resolved Hide resolved
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

❌ Changes requested.

  • Performed an incremental review on ceb5f53
  • Looked at 724 lines of code in 7 files
  • Took 1 minute and 43 seconds to review
More info
  • Skipped 0 files when reviewing.
  • Skipped posting 0 additional comments because they didn't meet confidence threshold of 50%.

Workflow ID: wflow_5RRPWZvQ9FmeU8EQ


Want Ellipsis to fix these issues? Tag @ellipsis-dev in a comment (or add @ellipsis-agent to your repo for autocomplete). We'll respond in a few minutes. Learn more here. You can customize Ellipsis with review rules, user-specific overrides, quiet mode, and more. See docs.

openapi.yaml Show resolved Hide resolved
@whiterabbit1983 whiterabbit1983 merged commit 9bcf5a9 into dev May 1, 2024
9 checks passed
@whiterabbit1983 whiterabbit1983 deleted the f/docs-search branch May 1, 2024 05:58
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.

2 participants