-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
🐛 Bug Report: Large zip breaking stream endpoint #859
Comments
Looks like it happens because the file is not being chunked properly or at all when answering, thus resulting current context token overload |
Hi, I would like to work on this issue. |
When you upload a big zip file what answer should it provide? |
Just shouldn't break. Basically make sure that it doesn't error out. |
I am interested to work on this issue. |
The issue is with the implementation of rst parser, in each file, it looks for a header and a text below it, but for the zip file we are testing on, it is just a single file with no header available, hence it is not being chunked. This header and text breakdown thing also seems to be an issue for markdown parser. The file should be chunked based on tokens or bytes and this tuple implementation also need to be updated. |
Yeah seems like thats the issue, lets add another token size handler to it maybe? |
Hey, I have updated the code and created a PR, can you review it and approve, I am new to open source contributions and do not know how it works, after making a PR. Open to feedbacks. |
@dartpain Can you review my changes and provide feedback, and approve if implementation seems correct. |
📜 Description
Stream endpoint doesn't provide answer when embedded file in zip archive is long.
👟 Reproduction steps
docs_tester.zip
👍 Expected behavior
DocsGPT should provide an answer.
👎 Actual Behavior with Screenshots
No answer, stream endpoint breaks.
💻 Operating system
MacOS
What browsers are you seeing the problem on?
Chrome
🤖 What development environment are you experiencing this bug on?
Docker
🔒 Did you set the correct environment variables in the right path? List the environment variable names (not values please!)
No response
📃 Provide any additional context for the Bug.
No response
📖 Relevant log output
No response
👀 Have you spent some time to check if this bug has been raised before?
🔗 Are you willing to submit PR?
None
🧑⚖️ Code of Conduct
The text was updated successfully, but these errors were encountered: