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

Attachment store for assistant2 #11327

Merged
merged 12 commits into from
May 3, 2024
Merged

Attachment store for assistant2 #11327

merged 12 commits into from
May 3, 2024

Conversation

rgbkrk
Copy link
Member

@rgbkrk rgbkrk commented May 2, 2024

This sets up a way for the user (or Zed) to push context instead of having the model retrieve it with a function. Our first use is the contents of the current file.

image image

I heard the asst2 example was deleted in another branch so I deleted that here too since we wanted the workspace access.

Release Notes:

  • N/A

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label May 2, 2024
@rgbkrk rgbkrk force-pushed the asst2-push-contexts branch 2 times, most recently from 36a81ba to b6352d4 Compare May 3, 2024 13:34
@rgbkrk rgbkrk changed the title Context store for assistant2 Attachment store for assistant2 May 3, 2024
@rgbkrk rgbkrk merged commit 3e5dcd1 into main May 3, 2024
8 checks passed
@rgbkrk rgbkrk deleted the asst2-push-contexts branch May 3, 2024 21:48
maxbrunsfeld added a commit that referenced this pull request May 4, 2024
Chat message rendering was restructured in
#11327, but it caused tool
calls not to be rendered if the assistant hadn't generated any message
text.

before:

![Screenshot 2024-05-03 at 10 02
57 PM](https://github.com/zed-industries/zed/assets/326587/2b7fd763-0c75-4690-9824-3bd37a3efef2)

after:

<img width="518" alt="Screenshot 2024-05-03 at 11 17 45 PM"
src="https://github.com/zed-industries/zed/assets/326587/34de19ba-daf2-4ac1-9fe0-f51d0ce94872">

Release Notes:

- N/A
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed The user has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant