Skip to content

Conversation

@sinomoe
Copy link
Contributor

@sinomoe sinomoe commented Jun 12, 2024

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes # (issue)

Type of Change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update, included: Dify Document
  • Improvement, including but not limited to code refactoring, performance optimization, and UI/UX improvement
  • Dependency upgrade

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • manual test

Suggested Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • I ran dev/reformat(backend) and cd web && npx lint-staged(frontend) to appease the lint gods
  • optional I have made corresponding changes to the documentation
  • optional I have added tests that prove my fix is effective or that my feature works
  • optional New and existing unit tests pass locally with my changes

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. ⚙️ feat:model-runtime labels Jun 12, 2024
Copy link
Member

@laipz8200 laipz8200 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great!
Thanks for your work.

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jun 12, 2024
@laipz8200 laipz8200 merged commit 0ce97e6 into langgenius:main Jun 12, 2024
@takatost takatost mentioned this pull request Jun 14, 2024
@hulijiashaonian
Copy link

感谢你的努力!我试了一圈还是不知道怎么启用,我用doubao-pro-32k,依然只能是ReAct模式

@laipz8200
Copy link
Member

感谢你的努力!我试了一圈还是不知道怎么启用,我用doubao-pro-32k,依然只能是ReAct模式

We just released this new feature. Make sure to update your Dify to version 0.6.11.

dengpeng pushed a commit to dengpeng/dify that referenced this pull request Jun 16, 2024
HuberyHuV1 pushed a commit to HuberyHuV1/dify that referenced this pull request Jul 22, 2024
@qdaoming
Copy link

qdaoming commented Aug 4, 2024

@sinomoe I have tried Doubao-pro-32K in v0.6.11, but it failed with message:
"FunctionCallPostProcessError message: model generated content is invalid."
Have you met such issue.

@sinomoe
Copy link
Contributor Author

sinomoe commented Aug 5, 2024

@sinomoe I have tried Doubao-pro-32K in v0.6.11, but it failed with message: "FunctionCallPostProcessError message: model generated content is invalid." Have you met such issue.

It's an error generated by Ark, indicating that the model-generated content is not in a valid JSON format. Since dify used ark v2 api endpoint, there might be some compatibility issue, i will look into this ASAP

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants