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

IMemoryStore.RemoveAsync

Open EachShow opened this issue 1 year ago • 2 comments

Describe the bug An error will occur when call IMemoryStore.RemoveAsync

Expected behavior Deserialization of types without a parameterless constructor, a singular parameterized constructor, or a parameterized constructor annotated with 'JsonConstructorAttribute' is not supported. Type 'Microsoft.SemanticKernel.Connectors.Memory.Qdrant.Http.ApiSchema.QdrantResponse'. Path: $ | LineNumber: 0 | BytePositionInLine: 1.

Desktop (please complete the following information):

  • OS: Windows
  • IDE: Visual Studio 2022
  • NuGet Package Version 0.13.442.1-preview

Additional context the vector database is Qdrant

EachShow avatar May 12 '23 06:05 EachShow

@tawalke any thoughts here?

craigomatic avatar May 16 '23 21:05 craigomatic

Let me fix this?

EachShow avatar May 18 '23 02:05 EachShow

Closing as we ended up fixing this over at #1025 - thank you for the PR in any case :)

craigomatic avatar Jun 02 '23 23:06 craigomatic

image Hi @craigomatic, I am still encountering an error with version 0.19.230804.2-preview in NuGet. Has the fix been released for this version?

Sam--Shan avatar Aug 11 '23 01:08 Sam--Shan

image Hi @craigomatic, I am still encountering an error with version 0.19.230804.2-preview in NuGet. Has the fix been released for this version?

Hmm, it was fixed in a prior version, maybe a regression has snuck in. If you are using Qdrant, internally it's doing deserialisation via https://github.com/microsoft/semantic-kernel/blob/4ea1d48dfcbe0d1594351b6d946d60f2b9dd60a6/dotnet/src/Connectors/Connectors.Memory.Qdrant/QdrantVectorDbClient.cs#L203

Would you mind opening a new issue and we will investigate?

craigomatic avatar Aug 11 '23 20:08 craigomatic