I propose creating a BaseResultsStorage class to serve as the base class for both PineconeResultsStorage and DefaultResultsStorage. This would allow us to automatically generate instances based on the provided configuration file, unifying the interfaces and improving the overall user experience.
By implementing this change, users can focus on the process without needing to know which specific storage instance is being used behind the scenes. This approach ensures a consistent and intuitive experience across different storage instances.
Please let me know your thoughts on this proposal, and if there are any further considerations or refinements needed.
I propose creating a BaseResultsStorage class to serve as the base class for both PineconeResultsStorage and DefaultResultsStorage. This would allow us to automatically generate instances based on the provided configuration file, unifying the interfaces and improving the overall user experience.
By implementing this change, users can focus on the process without needing to know which specific storage instance is being used behind the scenes. This approach ensures a consistent and intuitive experience across different storage instances.
Please let me know your thoughts on this proposal, and if there are any further considerations or refinements needed.