-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Migration of experimental ChatMessage
to Haystack
#8583
Comments
MotivationDuring the Tools experiment, we have refactored the
Existing Haystack dataclass; Experimental dataclass Now, we plan to port these changes into Haystack. My impression is that introducing the new dataclass without breaking changes is rather difficult. I would prefer to minimize breaking changes, but make it clear and transparent to users what is changing. Breaking changesReplacing the old dataclass with the new one will introduce the following breaking changes:
Existing class methods to create Some breaking changes we can avoid/smoothly introduce
|
@anakin87 out of curiosity do the changes discussed also allow for support of Image message types? For example, the Image type used by OpenAI https://github.com/openai/openai-python/blob/83f4774156dc3e29c7fe6be9ffd681df68534509/src/openai/types/image.py#L10 This would be needed to be able to pass images to ChatGenerators to run Visual QA. |
@sjrl we will not immediately introduce support for images with the changes in 2.9.0, but that is the idea: make content more flexible. Images will be just another type of content. |
Summary and motivation
Plan
Tasks
ChatMessage
- introducetext
property #8587ChatMessage
#8654ChatMessage
dataclass #8623ChatMessage
- remove deprecatedfunction
role andChatMessage.from_function
#8653The text was updated successfully, but these errors were encountered: