Skip to content
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

Modification prompt update #357

Merged
merged 5 commits into from
Sep 27, 2024
Merged

Conversation

Hello-zoka
Copy link
Collaborator

Improving the prompt to reduce occurrences of problems like in #252

@Hello-zoka Hello-zoka added bug Something isn't working improvement New feature labels Sep 7, 2024
@Hello-zoka Hello-zoka added the Ready for review PR redy for review label Sep 9, 2024
@pderakhshanfar pderakhshanfar added the In progress PR is in progress label Sep 18, 2024
val prompt = buildString {
append("For this test:\n ```\n ")
append(testCase)
append("\n```\nYou are allowed to generate only 1 test method. Do not change class and method names.")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can make it more assertive + less tokens:

Generate/Output a SINGLE test case/method.

@Hello-zoka Hello-zoka removed the In progress PR is in progress label Sep 26, 2024
@Vladislav0Art
Copy link
Collaborator

Does it close any issue from the issue list?

@Hello-zoka
Copy link
Collaborator Author

Does it close any issue from the issue list?

No. It just improves LLM behavior to reduce the number of such problems as in #252. To fully resolve this issue, we have to add smth like a feedback cycle or proper error notification to user. Unfortunately, both of the solutions require many changes in the core module, so we've decided not to include them in the upcoming release

@Hello-zoka Hello-zoka merged commit 9766597 into development Sep 27, 2024
3 checks passed
@arksap2002 arksap2002 deleted the izaitsev/bugs/modification-prompt branch September 30, 2024 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working improvement New feature Ready for review PR redy for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants