Skepticism towards the Necessity of ConversationBufferMemory: Combining with ConversationBufferWindowMemory for Better Memory Management

[fritzprix]

I'm highly skeptical if ConversationBufferMemory is actually needed compared to ConversationBufferWindowMemory. There are two main issues with it:

1. As usage continues, the list in chat_memory will become ever-increasing. (actually this is common for both at the moment, seems very weired though)
2. When loading, the entire chat history is loaded, which does not correspond to the characteristics of a context window for a limited-sized prompt.

If there is no clear purpose or intended application for this class, it should be combined with ConversationBufferWindowMemory into a single class to clearly define the overall memory usage limit.