Skip to content
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

event: tts_message_end always return in api streaming response_mode #8843

Closed
5 tasks done
xmenw opened this issue Sep 27, 2024 · 4 comments
Closed
5 tasks done

event: tts_message_end always return in api streaming response_mode #8843

xmenw opened this issue Sep 27, 2024 · 4 comments
Labels
🐞 bug Something isn't working

Comments

@xmenw
Copy link

xmenw commented Sep 27, 2024

Self Checks

  • This is only for bug report, if you would like to ask a question, please head to Discussions.
  • I have searched for existing issues search for existing issues, including closed ones.
  • I confirm that I am using English to submit this report (我已阅读并同意 Language Policy).
  • [FOR CHINESE USERS] 请务必使用英文提交 Issue,否则会被关闭。谢谢!:)
  • Please do not modify this template :) and fill in all the required fields.

Dify version

0.8.3

Cloud or Self Hosted

Cloud

Steps to reproduce

curl --location 'https://api.dify.ai/v1/chat-messages'
--header 'Authorization: Bearer app-PqAVQAqIr7VJgxQTN3pIa9Oe'
--header 'Content-Type: application/json'
--data '{
"inputs": {},
"query": "What are the specs of the iPhone 13 Pro Max?",
"response_mode": "streaming",
"conversation_id": "",
"user": "abc-123"
}'

image

✔️ Expected Behavior

This parameter is not returned if text_to_speech is not configured and disable is enabled

❌ Actual Behavior

image

Copy link

dosubot bot commented Sep 27, 2024

Hey @xmenw! I'm here to help you with any bugs, questions, or contributions you have for Dify. I can assist you while you wait for a human maintainer. Ask me anything!

The "tts_message_end" event always returns in the API streaming response_mode even when text_to_speech is not configured and disabled because it is a predefined event in the streaming response structure. This event is used to indicate the end of the audio stream, and its presence in the response does not depend on the actual configuration of text_to_speech [1].

To continue talking to Dosu, mention @dosu.

@dosubot dosubot bot added the 🐞 bug Something isn't working label Sep 27, 2024
@crazywoola
Copy link
Member

crazywoola commented Sep 27, 2024

@ic-xu Could you take a look at this.

@ic-xu
Copy link
Contributor

ic-xu commented Sep 27, 2024

@ic-xu你能看看这个吗?

OK, I'll just filter it.

ic-xu added a commit to ic-xu/dify that referenced this issue Sep 27, 2024
@ic-xu
Copy link
Contributor

ic-xu commented Sep 27, 2024

PR. #8846

ZhouhaoJiang added a commit that referenced this issue Sep 29, 2024
* main: (77 commits)
  feat: add voyage ai as a new model provider (#8747)
  docs: add english versions for the files customizable_model_scale_out and predefined_model_scale_out (#8871)
  fix: #8843 event: tts_message_end always return in api streaming resp… (#8846)
  Add Jamba and Llama3.2 model support (#8878)
  fix(workflow): update tagging logic in GitHub Actions (#8882)
  chore: bump ruff to 0.6.8 for fixing violation in SIM910 (#8869)
  refactor: update Callback to an abstract class (#8868)
  feat: deprecate gte-Qwen2-7B-instruct embedding model (#8866)
  feat: add internlm2.5-20b and qwen2.5-coder-7b model (#8862)
  fix: customize model credentials were invalid despite the provider credentials being active (#8864)
  fix: update qwen2.5-coder-7b model name (#8861)
  fix(workflow/nodes/knowledge-retrieval/use-config): Preserve rerankin… (#8842)
  chore: fix wrong VectorType match case (#8857)
  feat: add min-connection and max-connection for pgvector (#8841)
  feat(Tools): add feishu tools (#8800)
  fix: delete harm catalog settings for gemini (#8829)
  Add Llama3.2 models in Groq provider (#8831)
  feat: deprecate mistral model for siliconflow (#8828)
  fix: AnalyticdbVector retrieval scores (#8803)
  fix: close log status option raise error (#8826)
  ...
ZhouhaoJiang added a commit that referenced this issue Sep 29, 2024
* feat/new-login: (79 commits)
  feat: add setup only
  feat: remove self-host
  feat: add voyage ai as a new model provider (#8747)
  docs: add english versions for the files customizable_model_scale_out and predefined_model_scale_out (#8871)
  fix: #8843 event: tts_message_end always return in api streaming resp… (#8846)
  Add Jamba and Llama3.2 model support (#8878)
  fix(workflow): update tagging logic in GitHub Actions (#8882)
  chore: bump ruff to 0.6.8 for fixing violation in SIM910 (#8869)
  refactor: update Callback to an abstract class (#8868)
  feat: deprecate gte-Qwen2-7B-instruct embedding model (#8866)
  feat: add internlm2.5-20b and qwen2.5-coder-7b model (#8862)
  fix: customize model credentials were invalid despite the provider credentials being active (#8864)
  fix: update qwen2.5-coder-7b model name (#8861)
  fix(workflow/nodes/knowledge-retrieval/use-config): Preserve rerankin… (#8842)
  chore: fix wrong VectorType match case (#8857)
  feat: add min-connection and max-connection for pgvector (#8841)
  feat(Tools): add feishu tools (#8800)
  fix: delete harm catalog settings for gemini (#8829)
  Add Llama3.2 models in Groq provider (#8831)
  feat: deprecate mistral model for siliconflow (#8828)
  ...
hgnulb pushed a commit to hgnulb/dify that referenced this issue Oct 2, 2024
cuiks pushed a commit to cuiks/dify that referenced this issue Oct 8, 2024
lau-td pushed a commit to heydevs-io/dify that referenced this issue Oct 23, 2024
idonotknow pushed a commit to AceDataCloud/Dify that referenced this issue Nov 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants