-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
arch: deprecating recall action and search_memory
#2900
Conversation
@@ -161,6 +161,25 @@ parse_pptx(file_path: str) -> None: | |||
Args: | |||
file_path: str: The path to the file to open. | |||
|
|||
parse_audio(file_path: str, model: str = 'whisper-1') -> None: |
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.
Your local environmental variables seem to mess this up.
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.
Can you remove OPENAI_API_KEY
and OPENAI_BASE_URL
? I'll add an unset to the regenerate.sh later
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.
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.
merged! now re-generating integration tests now!
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.
Sorry, but you have to do a search (pun intended) for search_memory
in the codebase to also remove that one (~9 files). 😬
Other than that, this looks good.
@tobitege done! more than happy to! |
search_memory
Great! The codebase smells so much nicer now with all that baggage gone! 🤣 |
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, great job!
What is the problem that this fixes or functionality that this introduces? Does it fix any open issues?
Remove
RecallAction
to clean up the codebase and makeRuntime
more concise, since the current implementation isRecall
is not really working.I think @enyst's memory representation + condensation idea (#2709, #2021) is probably better to implement both
recall
andsummary
, other than putting them intoRuntime
class?Give a brief summary of what the PR does, explaining any non-trivial design decisions
Other references