FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

feat: update retrieval and assistant by Dttbd · Pull Request #55 · TaskingAI/TaskingAI-Python-Client · GitHub

/ TaskingAI-Python-Client Public

feat: update retrieval and assistant - #55

Merged
jameszyao merged 1 commit into
masterfrom
feat/record_and_chat_update
Nov 1, 2024
ConversationCommits (1)ChecksFiles changed
Merged

feat: update retrieval and assistant#55
jameszyao merged 1 commit into
masterfrom
feat/record_and_chat_update

Conversation

Dttbd commented Nov 1, 2024 •
edited by jameszyao
Loading

Copy link
Copy Markdown
Contributor

Pull Request

PR Description

  1. add new collection type: QA
  2. remove depreciated models and apis

Type of Change

  • Bug fix
  • New feature
  • Performance enhancement
  • Code refactor
  • Documentation update
  • Other, please describe:

Checklist

Before submitting this PR, please make sure:

  • I have read the CONTRIBUTING.md guidelines.
  • I have tested my changes locally to ensure they are effective.
  • I have updated the necessary documentation (if applicable).

jameszyao reviewed Nov 1, 2024
View reviewed changes
Comment thread examples/crud/tool_crud.ipynb Outdated
jameszyao reviewed Nov 1, 2024
View reviewed changes
Comment thread taskingai/assistant/assistant.py Outdated
jameszyao reviewed Nov 1, 2024
View reviewed changes
Comment thread taskingai/client/apis/__init__.py Outdated
jameszyao reviewed Nov 1, 2024
View reviewed changes
Comment thread taskingai/client/models/entities/collection.py Outdated
Dttbd force-pushed the feat/record_and_chat_update branch from 6143a4a to 58cbf4f Compare November 1, 2024 10:50
jameszyao force-pushed the feat/record_and_chat_update branch from 7cba8e7 to fe30c0c Compare November 1, 2024 11:41
jameszyao requested review from jameszyao and removed request for taskingaijc November 1, 2024 14:46
jameszyao approved these changes Nov 1, 2024
View reviewed changes

jameszyao left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM

jameszyao merged commit 563da8a into master Nov 1, 2024
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

jameszyao jameszyao approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

Back | FazBrowse Home | New Git URL