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

Feature - UI for AI Chat #587

Merged
merged 8 commits into from
Oct 20, 2024
Merged

Conversation

abhiroopc84
Copy link
Contributor

Description

Implemented the AI Chat UI as mentioned in #550. I've increased the width of the EditPanel to accommodate the UI elements better, we can decide on a default width once the resizable EditPanel implementation is done.

image
image

What is the purpose of this pull request?

  • New feature
  • Documentation update
  • Bug fix
  • Refactor
  • Release
  • Other

@drfarrell
Copy link
Collaborator

I did an initial pass through just tidying some stuff up, but I think we'll have a better idea of what the next phase of this will be when we have some of the first AI features together. @Kitenite what do you think?

@abhiroopc84
Copy link
Contributor Author

Which model are you planning to use for the AI features @drfarrell ?

@Kitenite
Copy link
Contributor

Kitenite commented Oct 19, 2024

@abhiroopc84 , I've used openai in the past and their API is always improving so likely that. I personally use claude so will give the docs a look before I lock down to one.

Possibly going to use an abstraction layer like langchain but spiking on some alternatives to that. I didn't like their API the last time I used it.

@abhiroopc84
Copy link
Contributor Author

@Kitenite are you planning on fine tuning the model to better perform for our use case? Last I checked, I think Claude does support fine tuning their model, but OpenAI didn't yet support it.

@Kitenite
Copy link
Contributor

@abhiroopc84 No plans as of yet. We'll start with some baseline performance, gather some feedback and decide how we want to proceed. There are a lot of different use cases we want to try before optimizing any particular one.

Copy link
Contributor

@Kitenite Kitenite left a comment

Choose a reason for hiding this comment

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

Oh man I'm excited for this! Nice work

@Kitenite Kitenite merged commit 8fa49af into onlook-dev:main Oct 20, 2024
@abhiroopc84 abhiroopc84 deleted the feat/ai-chat-ui branch October 21, 2024 14:19
@Kitenite Kitenite linked an issue Oct 25, 2024 that may be closed by this pull request
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.

[feat] UI for our AI Chat – no functionality required
3 participants