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

"Knowledge" block settings: The implementation and the documentation are different. #11976

Open
5 tasks done
TakahitoUsui opened this issue Dec 23, 2024 · 1 comment
Open
5 tasks done
Labels
📚 documentation Improvements or additions to documentation

Comments

@TakahitoUsui
Copy link

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.14.1

Cloud or Self Hosted

Self Hosted (Docker)

Steps to reproduce

I set the "Knowledge" in Dify "0.14.1".

  • Index Method: Economical
  • Retrieval Setting: Inverted Index

✔️ Expected Behavior

I hope when the workflow use one "Knowledge", the rerank model is optional.

❌ Actual Behavior

When I use "Knowledge Retrieval" block with this "Knowledge", Dify says "Rerank Model is required". But in this setting "Knowledge", the rerank model is optional. That's what is written in the Dify's documents.

image

@dosubot dosubot bot added the 📚 documentation Improvements or additions to documentation label Dec 23, 2024
Copy link

dosubot bot commented Dec 23, 2024

I found a similar closed issue titled Knowledge Retrieval Node Incorrectly Reverts to RerankModel, which was resolved by a pull request. The pull request addressed the issue by ensuring that the top_n parameter is valid and that the query is not empty, improving the user experience with the Wenxin Rerank model [1].

Regarding your current issue, it seems there is a discrepancy between the Dify documentation and the actual implementation in the codebase concerning the 'Knowledge Retrieval' block settings. The documentation suggests that the rerank model should be optional for the 'Economical' index method and 'Inverted Index' retrieval setting, but the implementation requires it. This might indicate a bug or an oversight in the documentation or codebase. Unfortunately, I couldn't find specific information about this discrepancy in the available data.

To continue talking to Dosu, mention @dosu.


Help Dosu learn! Give it feedback: Great Response | Irrelevant Answer | Incorrect Sources | Too Verbose | Hallucination | Other

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📚 documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

1 participant