-
Notifications
You must be signed in to change notification settings - Fork 5.5k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
feat: updated ChatOutput component that can accept Data, Dataframe and Message #6643
Conversation
https://www.loom.com/share/5d58495d46a14575b1dd383156013071?sid=c9977048-d299-4e1d-943c-c34dda96892c This PR might require improvements |
Needs better testing Cause this is a major change. |
@Cristhianzl can you take a look into the frontend tests? |
CodSpeed Performance ReportMerging #6643 will not alter performanceComparing Summary
|
@edwinjosechittilappilly The problem relates to the starter projects updates. The tests are expecting something that they are not finding because something changed the starter projects. |
This pull request includes several changes to the
ChatOutput
component to enhance its functionality and improve data handling. The most important changes include the addition of new input types, a new input field for data cleaning, and improved data conversion and validation methods.Enhancements to
ChatOutput
component:Data
andDataFrame
to theinput_value
field, allowing for more flexible data handling. [1] [2] [3]BoolInput
field namedclean_data
to enable basic data cleaning operations. [1] [2] [3]MessageInput
withHandleInput
for theinput_value
field to support multiple input types and ensure that the field is required. [1] [2]message_response
method to include data conversion to string and updated message properties handling. [1] [2]_validate_input
,_safe_convert
, andconvert_to_string
to validate and safely convert input data to string format, ensuring proper error handling.