openai / openai-dotnet

The official .NET library for the OpenAI API
https://www.nuget.org/packages/OpenAI
MIT License
707 stars 60 forks source link

Assistant with Multiple Vector Store Ids? #78

Closed hemnaathR closed 1 week ago

hemnaathR commented 1 week ago

Hello, I noticed that when creating the Assistant you could pass in a list of VectorStoreIds? At the moment however we can only pass in one VectorStoreId per Assistant. Does that mean there are plans for an Assistant to use multiple Vector Stores in the future?

Assistant assistant = await assistantClient.CreateAssistantAsync(
    model: "gpt-4",
    new AssistantCreationOptions()
    {
        Name = "Assistant v2",
        Tools =
                {
                    new FileSearchToolDefinition(),
                },
        ToolResources = new()
        {
            FileSearch = new()
            {
                VectorStoreIds = [vs1.Id] // can support multiple ids in the future?
            }
        },
    });
trrwilson commented 1 week ago

Great question, @hemnaathR! With the disclaimer that I have no insider knowledge one way or another, I'll say that guess makes a lot of sense to me and that I share it -- there wouldn't be much of a reason to define this as an "array of one item" (per API reference and YAML spec if there weren't a prediction of having more than one.

hemnaathR commented 1 week ago

Cool. Fingers crossed. 🤞🏼