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

סנכרון נפרד של רכיבי הבוט #5

Open
noamoss opened this issue Dec 24, 2024 · 1 comment
Open

סנכרון נפרד של רכיבי הבוט #5

noamoss opened this issue Dec 24, 2024 · 1 comment
Assignees

Comments

@noamoss
Copy link
Collaborator

noamoss commented Dec 24, 2024

משימת המשך ל-https://github.com/whiletrue-industries/botnim/issues/3.

המשימה נועדה לאפשר סנכרון נפרד [במקום הבניה מחדש בכל פעם, כפי שמומשה עד כה] של כל אחד מהבוטים. הרכיבים המדוברים הם:

  • הפרומפט (ההנחיות לבוט)
  • בסיס הידע הסטטי (למשל, חוקים או תקנון)
  • הידע התושבעי

בעקבות משימה זו ניתן יהיה לסנכרן את כל הרכיבים בבת אחת (מה שכרוך בזמן המתנה ארוך יותר ובעלויות ההקמה מחדש של בסיס הנתונים) או רק את הרכיבים הרצויים.

@akariv akariv self-assigned this Jan 5, 2025
@noamoss noamoss assigned noamoss and unassigned akariv Jan 9, 2025
@noamoss
Copy link
Collaborator Author

noamoss commented Jan 10, 2025

comments from previous round code review:

I think a better interface would have been something like:
--update-kb common-knowledge.md (kb as in knowledge-base)
This allows for a more flexible command line interface instead of hard coding a filename in your code.

We can even extend this to allow for two special values: none and all. none is the default, and it means "don't update knowledge bases if they already exist". all is the opposite (i.e. update all knowledge bases). This allows for one parameter instead of needing the --update-instructions parameter too And also deprecating --replace-context obviously

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants