After successfully adding chromaDB data, return the ID.

[lanbaoshen]

Why are these changes needed?

After creation, return the ID that can index the data, allowing direct data operations via the ID.

Related issue number

No related issue

Checks

• I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://github.com/microsoft/autogen/blob/main/CONTRIBUTING.md to build and test documentation locally.
• I've added tests (if relevant) corresponding to the changes introduced in this PR.
• I've made sure all auto checks have passed.

[victordibia]

Thanks for the contribution.

Can you provide more details on the use case you are exploring? e.g, what are some examples of "direct data operations via the ID."

Note that the base Memory class does not have a return type, and adding a return type breaks the abstraction. (please see the readme here on how to run tests.)

We should only do this when the use case is clear.

[lanbaoshen]

Thanks for the contribution.

Can you provide more details on the use case you are exploring? e.g, what are some examples of "direct data operations via the ID."

Note that the base Memory class does not have a return type, and adding a return type breaks the abstraction. (please see the readme here on how to run tests.)

We should only do this when the use case is clear.

Hi @victordibia , in my case, I want to implement a feature that automatically clears outdated memory data. For example, when I store data in ChromaDB, I will use the UUID and collection as keys in Redis and set a timeout. When Redis sends a keyevent expired message, I will directly delete the data in the collection using the UUID.

Instead of directly modifying the return type, perhaps introducing an optional parameter to allow users to customize IDs would be better?

Additionally, are there any plans to expand additional methods, such as upsert?