-
Notifications
You must be signed in to change notification settings - Fork 243
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
Escaped double quotes: \" in model responses get shown as just " in chat UI #437
Comments
@carlrobertoh I can't really find handling of escaped quotes or corresponding tests in llm-client 😅 There are multiple locations dealing with escape sequences (including quotes) in CodeGPT cpp files, but I can't find tests. |
@AlexanderLuck What service / model service are you using? |
I use Claude and GPT4, both have that issue. |
Just checked again, also happens with user inputs, if you just put in chat: var query = $"from(bucket: "{BucketName}") "; |
This bug is super annoying |
This will be fixed in the next release. I'm not yet entirely sure why it was added in the first place. |
What happened?
Hi,
when I let modesl generate code containing strings with escaped " the codeGPT UI seems to strip the \ from the text for example I get as an answer:
var query = $"from(bucket:"{BucketName}") |> range(start: -1y) |> filter(fn: (r) => r._measurement == "metadata_tracking" and r.measurement_name == "{measurementName}") |> last()";
Correct would be:
var query = $"from(bucket: \"{BucketName}\") |> range(start: -1y) |> filter(fn: (r) => r._measurement == \"metadata_tracking\" and r.measurement_name == \"{measurementName}\") |> last()";
When I ask the model to fix the first response it correctly tells me the delimiters are missing and posts exactly the same with missing delimiters.
Relevant log output or stack trace
No response
Steps to reproduce
Ask in chat:
generate a c# string log message with other variables using a " delimited string
CodeGPT version
2.5.1
Operating System
None
The text was updated successfully, but these errors were encountered: