Reverse prompt is sometimes ignored.

[tjohnman]

I haven't found a consistent pattern to reproduce this, but sometimes the model will continue outputting text even after it has printed the reverse prompt. If colors are enabled, they will change as if the new text was user input, but it is generated by the model. After this happen it might or might not revert to its proper behavior once it finds the reverse prompt again.

I have noticed the color change doesn't always happen right on the prompt, but sometimes it happens a few words before it. I don't know enough about how this code works yet to speculate, but in case this has something to do with parallelism, I'm using -t 16.

[tjohnman]

I think this might have to do with the way the program deals with a large -n token count.

Related to this, there is the issue of interactive mode dropping out back to the shell after the number of requested tokens has been consumed. This makes people use large -n numbers and put things in their prompts like “the conversation goes on forever” in order to try and avoid it.

I think we could handle running out of tokens when in interactive mode differently, similar to what alpaca.cpp does. Instead of closing the program, resetting the counter and going into interacting mode.

I will try to explore both issues later today and see if I can make a pull request.

[Piezoid]

This makes people use large -n numbers and put things in their prompts like “the conversation goes on forever” in order to try and avoid it.

For this, we could add a flag that forbid the sampler to draw control symbols like begin/end of stream.
I tried a hardcoded version of that and it works well. I found that banning the bos (begin of stream) token is also needed, because the model sometimes attempt to start a new document, effectively masking most of the past context.

[tjohnman]

This makes people use large -n numbers and put things in their prompts like “the conversation goes on forever” in order to try and avoid it.

For this, we could add a flag that forbid the sampler to draw control symbols like begin/end of stream. I tried a hardcoded version of that and it works well. I found that banning the bos (begin of stream) token is also needed, because the model sometimes attempt to start a new document, effectively masking most of the past context.

Masking past context might be an issue, but I think it's useful to sample end of stream symbols if the model decides a particular sequence should end early. The main issue here (which is very easy to solve), is that ending the stream (or running out of token budget) shouldn't mean we should close the program if we are in interactive mode, but rather let the user input more text.

I think I have figured out where the bug with the sampler not stopping when it should is. It seems the program keeps sampling new tokens regardless of the interactive state.

If I'm not mistaken, the conditions in main.cpp:937 and main.cpp:995 can be true at the same time, but while is_interacting == true, no sampling should be done, otherwise we are both sampling more tokens from the model and asking the user for input at the same time. These should be exclusive. Making sampling happen only when is_interacting is false should fix this bug.

I'll see if I can fix it, test it and make a pull request.

[LoganDark]

Related to this, there is the issue of interactive mode dropping out back to the shell after the number of requested tokens has been consumed. This makes people use large -n numbers and put things in their prompts like “the conversation goes on forever” in order to try and avoid it.

Interactive mode also ends when the model says [end of text] even though that makes no sense, that should just go back to asking for user input. #181

edit: apparently that PR has been merged some hours ago, whoops

[jarcen]

I mentioned it in another issue: llama was trained with tokenizer augmentations, means the tokenizer occasionally did sub-optimal word partitioning at training time:

https://github.com/google/sentencepiece#subword-regularization-and-bpe-dropout

It is claimed it improves generalization. As a side effect trained model now considers forming words with sub-optimal tokens. Reverse prompt is detected by matching lists of tokens. So, if you search for ['▁Your' , '▁Input'] and model outputs ['▁Your', '▁I', 'n', 'p', 'u', 't'] then it will look exactly same but there will be no match and user will not receive control. Correct implementation must track text outputted by the model and search in that range for reverse prompt by characters rather than by tokens.

[tjohnman]

I mentioned it in another issue: llama was trained with tokenizer augmentations, means the tokenizer occasionally did sub-optimal word partitioning at training time:

https://github.com/google/sentencepiece#subword-regularization-and-bpe-dropout

It is claimed it improves generalization. As a side effect trained model now considers forming words with sub-optimal tokens. Reverse prompt is detected by matching lists of tokens. So, if you search for ['▁Your' , '▁Input'] and model outputs ['▁Your', '▁I', 'n', 'p', 'u', 't'] then it will look exactly same but there will be no match and user will not receive control. Correct implementation must track text outputted by the model and search in that range for reverse prompt by characters rather than by tokens.

I can confirm this has happened to me several times.