Skip to content

Feat/gpt 4o search#3731

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

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

Conversation

@wwzeng1
Copy link
Copy Markdown
Contributor

@wwzeng1 wwzeng1 commented May 14, 2024

add gpt-4o search

@vercel
Copy link
Copy Markdown

vercel bot commented May 14, 2024

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
Copy link
Copy Markdown
Contributor

sweep-nightly bot commented May 21, 2024

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