dosubot[bot]
commented
2 days ago I found a similar unsolved discussion that might be relevant to your issue: Error in item: Invalid variable, which was last updated on September 23, 2024 [1].
Addressing Your Issues:
-
Enhancement Request: Support for
sys.filesin IF Conditions- Currently, the
sys.filesparameter is used to store images uploaded by users and requires enabling the image upload function in the 'Features' section of the application orchestration page [2]. However, it seems thatsys.filescannot be directly referenced within the IF condition. This enhancement would indeed be useful for workflows involving different content types.
- Currently, the
-
Bug Fix: 'TextPromptMessageContent' object has no attribute 'text'
- The error
'TextPromptMessageContent' object has no attribute 'text'occurs because theTextPromptMessageContentclass uses thedataattribute to store text data, nottext. Ensure that you are using thedataattribute to access the text content.
- The error
Here is the correct way to handle TextPromptMessageContent:
from your_module import TextPromptMessageContent, PromptMessageContentType
# Correct initialization
text_content = TextPromptMessageContent(type=PromptMessageContentType.TEXT, data="Your text here")
# Accessing the text data
print(text_content.data) # This will print "Your text here"In your workflow, make sure to use content.data instead of content.text when dealing with TextPromptMessageContent objects [3][4].
Implementing Branching Logic Based on Content Type:
To implement branching logic based on content type, you can use the IF/ELSE node to check for the presence of images or specific keywords in sys.query. Here is an example of how to set up such a workflow:
- IF/ELSE Node: Use this node to check if
sys.querycontains the keyword "image". - Branch for Image Content: If the condition is met, process the content using the visual VL model.
- Branch for Text Content: If the condition is not met, process the content using the standard LLM.
This approach allows you to handle different types of content within the same workflow [5][6].
By addressing these points, you should be able to enhance your workflow and resolve the current bug.
To continue talking to Dosu, mention @dosu.
crazywoola
svcvit
free-cx
laipz8200
Self Checks
Dify version
0.8.3
Cloud or Self Hosted
Self Hosted (Docker)
Steps to reproduce
I'm trying to implement a branching logic in the workflow where regular text content is processed by a standard large language model (LLM), and image content is processed by a visual VL model. However, the system parameter sys.files cannot be referenced within the if branch.
To work around this, I added a keyword "image" to sys.query. If this keyword is present, the workflow will use the visual VL model.
The workflow functions correctly during the first round of conversation. However, in the second round, which only involves text, an error occurs: Run failed: 'TextPromptMessageContent' object has no attribute 'text'.
Reference image for the first round of conversation:
Reference image for the second round of conversation:
✔️ Expected Behavior
Enhancement Request: I would like the if condition to support sys.files so that image-related tasks can be processed using different models. Bug Fix: I'm also seeking a fix for the current bug.
❌ Actual Behavior
No response