-
Notifications
You must be signed in to change notification settings - Fork 893
feat: add support for configurable prompt mapping #1416
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?
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1416 +/- ##
==========================================
+ Coverage 34.76% 43.12% +8.35%
==========================================
Files 94 107 +13
Lines 6342 7636 +1294
==========================================
+ Hits 2205 3293 +1088
- Misses 4046 4193 +147
- Partials 91 150 +59 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Signed-off-by: hadi2f244 <[email protected]>
|
I noticed that the @AlexsJones, Is it correct !? |
|
this sounds like an excelent idea. |
Signed-off-by: Alex Jones <[email protected]>
| Client *kubernetes.Client | ||
| Language string | ||
| AIClient ai.IAI | ||
| PromptMap map[string]string |
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.
What does this bit do? We pass the prompt map into result, err := a.getAIResultForSanitizedFailures(texts, promptTemplate)
Closes #842
📑 Description
This pull request introduces support for custom prompts in AI backends. Previously, the system relied on predefined prompts within
prompts.go. Now, users can define and override prompts (Ink8sgpt.yaml) based on their specific use cases.✅ Checks
ℹ Additional Information
The documentation needs to be updated to reflect the changes in the k8sgpt.yaml file and how to implement custom prompt.