Skip to content

Feat/gpt 4o search#3731

Open
wwzeng1 wants to merge 4 commits into
mainfrom
feat/gpt-4o-search
Open

Feat/gpt 4o search#3731
wwzeng1 wants to merge 4 commits into
mainfrom
feat/gpt-4o-search

Conversation

@wwzeng1

@wwzeng1 wwzeng1 commented May 14, 2024

Copy link
Copy Markdown
Contributor

add gpt-4o search

@vercel

vercel Bot commented May 14, 2024

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sweep-chat 🔄 Building (Inspect) Visit Preview 💬 Add feedback May 31, 2024 4:28pm
sweep-chat-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 31, 2024 4:28pm
sweep-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 31, 2024 4:28pm

@sweep-nightly

sweep-nightly Bot commented May 21, 2024

Copy link
Copy Markdown
Contributor

Sweep: PR Review

Authors: MartinYe1234, wwzeng1, kevinlu1248
The Pull Request looks good! Sweep did not find any issues.

.gitignore was not reviewed because our filter identified it as typically a non-human-readable (auto-generated) or less important file (e.g., dist files, package.json, images). If this is an error, please let us know.

@MartinYe1234 MartinYe1234 added the sweep Assigns Sweep to an issue or pull request. label May 31, 2024
@MartinYe1234

Copy link
Copy Markdown
Collaborator

test comment

Comment thread sweepai/core/chat.py
raise e_
return
return llm_stream()
def llm_stream_openai():

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.

The new llm_stream_openai function introduces complexity with threading and queue management, which needs thorough testing to ensure reliability.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sweep Assigns Sweep to an issue or pull request.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants