-
Notifications
You must be signed in to change notification settings - Fork 8.8k
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
doc: Added explanation of chunk_overlap to knowledge API #12247
doc: Added explanation of chunk_overlap to knowledge API #12247
Conversation
Commit message not up to the community standard. I would suggest you do some readings on commit message format and also properlly include description in the PR. |
I have updated the PR Summary to include a more detailed description of the changes. Regarding the commit message format, I will make sure to follow the community standards for future commits. |
Hello there, please update the docs in JA as well. |
Thank you for your feedback. Currently, there is no documentation in Japanese for the Knowledge API. Therefore, I don't think there is any need to take action. If you think adding it would be helpful, feel free to create a new issue—or let me know, and I can create one. |
2eab1cc
to
ddbff19
Compare
Thanks :) Actually I figure it would be helpful to add JA document to this section, because we have added several templates in API develop section right now. We appreciate that you could add this if you have some spare time. |
Thank you.
I added a Japanese-language issue to the Knowledge API. |
Summary
The following four Knowledge APIs allow you to specify "chunk_overlap" as a parameter when creating a document.
However, this is not described in the API specifications, so it needs to be added.
Fixes #12180
Tip
Close issue syntax:
Fixes #<issue number>
orResolves #<issue number>
, see documentation for more details.Screenshots
Checklist
Important
Please review the checklist below before submitting your pull request.
dev/reformat
(backend) andcd web && npx lint-staged
(frontend) to appease the lint gods