serge icon indicating copy to clipboard operation
serge copied to clipboard

Added Detection against self responses

Open DVSProductions opened this issue 2 years ago • 5 comments

The models like writing "### Instruction:" from time to time. with this modification their generation gets interrupted as soon as they type ### effectively stopping them from generating their own prompts. Because this kills the llama process this also saves resources and isn't just a visual change.

Here is an example response before this change: image

Here is the same prompt and the (disabled) debug output from docker showing the changes in action:

Unbenannt

DVSProductions avatar Apr 05 '23 17:04 DVSProductions

If https://github.com/nsarrazin/serge/pull/143 is merged in, then your generate.py changes won't be valid

voarsh2 avatar Apr 05 '23 22:04 voarsh2

@DVSProductions Can you fix the merge conflicts?

gaby avatar Apr 17 '23 01:04 gaby

I've resolved the merge conflicts but the Docker push seems to fail with a 403? don't know that to do about that

DVSProductions avatar Apr 17 '23 15:04 DVSProductions

@DVSProductions Seems like another bug, for some reason it fails with external branches

gaby avatar Apr 18 '23 00:04 gaby

@DVSProductions I will merge this soon, trying to fix a last little bug with the CI :-)

gaby avatar Apr 22 '23 03:04 gaby

@gaby How are things going?

fishscene avatar May 18 '23 14:05 fishscene

@fishscene This branch has changes that are not compatible with the migration to Redis.

It's up to @DVSProductions to fix the merge conflicts

gaby avatar May 18 '23 14:05 gaby

Thanks for the update, mate!

fishscene avatar May 19 '23 13:05 fishscene

@DVSProductions Do you want to fix the conflicts or close this?

gaby avatar May 23 '23 22:05 gaby