-
Notifications
You must be signed in to change notification settings - Fork 6.8k
Enhanced llm editor #9174
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
base: main
Are you sure you want to change the base?
Enhanced llm editor #9174
Conversation
I have completed the modification |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Overall seems good to me!
Only concern now is that it changes the existing prompt which @leandermaben is running experiments on. Would appreciate a quick 👀 from @leandermaben .
An alternative way to get this merge faster would be keep the USER_MSG unchanged, and just add the CORRECT_SYS_MSG
OK, I have completed the modification. Please review it again. keep the USER_MSG unchanged, and just add the CORRECT_SYS_MSG |
Thanks for the updates. I will review it by evening today, apologies for the delay. |
Thanks 😃😃😃 |
@leandermaben There is a conflict, Need me to resolve it? |
Sure, that would be great! |
I have completed |
End-user friendly description of the problem this fixes or functionality this introduces.
When enable_auto_lint=true is set, the accuracy of code modification in llm editor is enhanced
Summarize what the PR does, explaining any non-trivial design decisions.
I modified edit.py to enhance this feature:
Link of any specific issues this addresses:
#9173