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.

Sign up for GitHub

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

Jump to bottom

reformat agentQnA sample guide #219

Merged
merged 2 commits into from
Nov 14, 2024
Merged

reformat agentQnA sample guide #219

merged 2 commits into from
Nov 14, 2024

Conversation

abdulari
Copy link
Contributor

this PR relates to #150

@chickenrae
Copy link
Member

@abdulari Are you taking on issue #150 or simply reformating?

@xuechendi
Copy link
Collaborator

is there a reason we only provide xeon doc, how about Gaudi?

@minmin-intel
Copy link

@abdulari We need to provide guides for deployment on Gaudi to use open-source LLMs. Currently, deployment on Xeon requires using OpenAI GPT-4o model API.

tomlenth
tomlenth approved these changes Nov 4, 2024
View reviewed changes
Copy link
Collaborator

@tomlenth tomlenth left a comment •
edited
Loading

Choose a reason for hiding this comment

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

approving.

@yinghu5 yinghu5 self-requested a review November 14, 2024 02:37
yinghu5
yinghu5 approved these changes Nov 14, 2024
View reviewed changes
Copy link
Collaborator

@yinghu5 yinghu5 left a comment

Choose a reason for hiding this comment

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

merged it and then update later

@yinghu5 yinghu5 merged commit 24b6ad9 into opea-project:main Nov 14, 2024
4 checks passed
@yinghu5 yinghu5 added this to the v1.1 milestone Nov 14, 2024
alexsin368 pushed a commit to alexsin368/opea-project-docs that referenced this pull request Nov 14, 2024
@abdulari @yinghu5 @alexsin368
reformat agentQnA sample guide (opea-project#219)
c54860a 
Signed-off-by: abdulari <[email protected]>
Co-authored-by: Ying Hu <[email protected]>
Signed-off-by: alexsin368 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xuechendi xuechendi xuechendi left review comments

@yinghu5 yinghu5 yinghu5 approved these changes

@tomlenth tomlenth tomlenth approved these changes

@chensuyue chensuyue Awaiting requested review from chensuyue chensuyue is a code owner

@ftian1 ftian1 Awaiting requested review from ftian1 ftian1 is a code owner

@mkbhanda mkbhanda Awaiting requested review from mkbhanda mkbhanda is a code owner

@preethivenkatesh preethivenkatesh Awaiting requested review from preethivenkatesh preethivenkatesh is a code owner

@chickenrae chickenrae Awaiting requested review from chickenrae chickenrae is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v1.1
Development

Successfully merging this pull request may close these issues.
6 participants
@abdulari @chickenrae @xuechendi @minmin-intel @tomlenth @yinghu5