semantic-kernel icon indicating copy to clipboard operation
semantic-kernel copied to clipboard

GitHub Repo Q&A Bot sample app error

Open pauldotyu opened this issue 1 year ago • 5 comments

Describe the bug Attempt to download the microsoft/semantic-kernel repo using the "GitHub Repo Q&A Bot" sample app results in an error with the message: "Bad Request => Positive number required. (Parameter 'bufferSize') Actual value was 0.

To Reproduce Steps to reproduce the behavior:

  1. Clone the repo using macOS m1
  2. Start the KernelHttpServer using func start --csharp command
  3. Copy .env.example as .env and populate with Azure OpenAI completion and embeddings model details (as defaults)
  4. Run yarn install && yarn start to start the sample app
  5. Go through the setup (values should be defaulted)
  6. Enter GitHub Respoitory URL: https://github.com/microsoft/semantic-kernel
  7. Enter Branch Name: main

Expected behavior Embeddings should be downloaded

Screenshots image

Desktop (please complete the following information):

  • OS: macOS (M1)
  • IDE: VS Code
  • NuGet Package Version [e.g. 0.1.0]

Additional context

NodeJS and Azure Function Core Tools versions.

$ node -v                   
v18.7.0

$ func --version     
4.0.5095

pauldotyu avatar Apr 17 '23 17:04 pauldotyu

Which model are you using for embeddings? ada-002?

craigomatic avatar Apr 18 '23 00:04 craigomatic

Which model are you using for embeddings? ada-002?

Correct. I've deployed the text-embedding-ada-002 model and set the deployment name to be the same name as the model.

pauldotyu avatar Apr 18 '23 14:04 pauldotyu

@pauldotyu thanks for bringing this up, we have filed a bug on this and will take a look to see what is going on with it.

evchaki avatar Apr 18 '23 20:04 evchaki

@craigomatic can you take a look at this

evchaki avatar May 16 '23 20:05 evchaki

@matthewbolanos can you take a look and see if this repros for you?

evchaki avatar May 25 '23 20:05 evchaki

I tried the sample with gpt-3.5-turbo and text-embedding-ada-002 and was not able to repro. From what I can tell, the KernelHttpServer has seen some changes in the last few weeks which may have addressed the underlying bug.

@pauldotyu, would you be able to share if the problem persists with the latest version?

matthewbolanos avatar Jun 04 '23 17:06 matthewbolanos

Closing this for now, please reopen if needed.

craigomatic avatar Jun 14 '23 16:06 craigomatic