Fix prefix/middle/suffix order

huggingface / llm-ls

LSP server leveraging LLMs for code completion (and more?)

Apache License 2.0

602 stars 49 forks source link

Fix prefix/middle/suffix order #47

Closed Maddimax closed 10 months ago

Maddimax commented 10 months ago

Looks to me like the order is wrong?

rhelmot commented 2 months ago

Hey, I just ran across this. Why was this closed? Applying this diff causes the system to actually generate sane suggestions when using deepseek-coder-v2. Does this need to be an option; are most models trained to accept <prefix> text <suffix> suffix <middle>?