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

Prevent web browser error and chat history not saving #10711

Closed
wants to merge 1 commit into from

Conversation

M-Allahham
Copy link

@M-Allahham M-Allahham commented Dec 7, 2024

Make sure to read the contributing guidelines before submitting a PR

When using bartowski/QwQ-32B-Preview-GGUF off huggingface, I would experience a console error about reading choices[0] at the end of the text generation. It would seem that chunk.data.choices doesn't exist at all!
This is adding an if statement to ensure that the member exists before appending to the chat's content.

Also removed chunk.data.stop as it was unused in the JS.

@ngxson
Copy link
Collaborator

ngxson commented Dec 8, 2024

It should already be fixed in the latest version, so this is not needed anymore.

@ngxson ngxson closed this Dec 8, 2024
@M-Allahham
Copy link
Author

It should already be fixed in the latest version, so this is not needed anymore.

Sorry, I pulled from master yesterday and encountered this issue. You can see in my own branch that it's 1 commit ahead and (currently) 3 behind the remote branch, where the 3 commits aren't related to this fix.
Can you point me to the latest version where this has been fixed?

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

Successfully merging this pull request may close these issues.

2 participants