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

Generation should stop after two new lines if that is the stop criteria #717

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Vidyaranya
Copy link

This addresses Issue 642. When the stop token is \n\n the generation should stop after generation two new lines.

Check the previous token that is generated and if it is repeated, and if the stop token contains this as the criteria, stop generating further tokens

@suchenzang
Copy link
Contributor

suchenzang commented Apr 30, 2023

Rebase on main, and tests will probably pass!

Summary: This addresses Issue 642. When the stop token is \n\n the generation
should stop after generation two new lines.
@Vidyaranya
Copy link
Author

Rebase on main, and tests will probably pass!

They still seem to fail!

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.

None yet

4 participants