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

improve web search #1537

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

improve web search #1537

wants to merge 13 commits into from

Conversation

flozi00
Copy link
Contributor

@flozi00 flozi00 commented Oct 21, 2024

This PR introduces multi query web search.
Long term goal to get perplexity like performance

@flozi00
Copy link
Contributor Author

flozi00 commented Oct 21, 2024

image

@flozi00
Copy link
Contributor Author

flozi00 commented Oct 22, 2024

@nsarrazin

Copy link
Collaborator

@nsarrazin nsarrazin left a comment

Choose a reason for hiding this comment

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

Overall I like the feature, if the max number of queries could be configurable I think it could be a nice addition!

The long-term goal I think would be that the model knows whether or not it needs to ask for another query, since right now we will always ask for N queries but this is a nice start!

src/lib/server/websearch/search/search.ts Outdated Show resolved Hide resolved
@flozi00
Copy link
Contributor Author

flozi00 commented Oct 23, 2024

Overall I like the feature, if the max number of queries could be configurable I think it could be a nice addition!

The long-term goal I think would be that the model knows whether or not it needs to ask for another query, since right now we will always ask for N queries but this is a nice start!

model now generates dynamic list of queries, filtered at the end for the num searches set by env to keep your bills low ;)

Add a new tool to fetch weather data using the Open-Meteo API.

* **New Tool Implementation**
  - Create `weather.ts` to define the weather tool with inputs and call method to fetch weather data.
  - Implement `call` method to fetch coordinates and weather data using utility functions.

* **Tool Integration**
  - Import and add the weather tool to the list of tools in `index.ts`.

* **Output Handling**
  - Add a new output component for weather data in `outputs.ts`.

* **Utility Functions**
  - Add utility functions in `utils.ts` to fetch weather data and coordinates from the Open-Meteo API.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/flozi00/chat-ui?shareId=XXXX-XXXX-XXXX-XXXX).
Add weather tool to fetch weather data
@flozi00
Copy link
Contributor Author

flozi00 commented Oct 24, 2024

image

Ups, added the commits by mistake to the branche
Now a weather tool is included here too

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