Karol Bajko comments

Results 8 comments of


                                            Karol Bajko

`warning: literal string will be frozen in the future` in ruby 3.4.0-preview2

Any update on that? Ruby 3.4 is publicly available now.

Opening the Assistant to accept custom LLM

Hi @andreibondarev, I think the Assistant needs a wrapped LLM object with a consistent interface that can handle all necessary actions. I'm not sure if the Assistant should know that...

Opening the Assistant to accept custom LLM

Hey @andreibondarev, I've changed a little an approach by allowing to inject custom LLM adapter when Assistant is initialized. That brings more flexibility to use it with totally custom LLM...

Opening the Assistant to accept custom LLM

Hey @andreibondarev, I’m required to use a closed wrapper built on top of multiple LLMs. It includes some policies to prevent data leaks, etc. The abstraction that the Assistant provides...

Opening the Assistant to accept custom LLM

@andreibondarev I've rebased with main branch. It simplified my PR now. WDYT about adding a custom LLM adapter through constructor?

Opening the Assistant to accept custom LLM

> > @andreibondarev I've rebased with main branch. It simplified my PR now. WDYT about adding a custom LLM adapter through constructor? > > So in order to make it...

Opening the Assistant to accept custom LLM

@andreibondarev I've implemented the latest concept. WDYT?

Opening the Assistant to accept custom LLM

@andreibondarev thanks for your feedback. Opening an adapter's factory is an option too. I have something like that: ```ruby module Langchain class Assistant module LLM class Adapter class