-
Notifications
You must be signed in to change notification settings - Fork 6
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
chore: update test_tools to read a file instead of get a password #38
Conversation
Signed-off-by: Adrian Cole <adrian.cole@elastic.co>
|
||
response = ex.reply() | ||
|
||
# It's possible this can be flakey, but in experience so far haven't seen it |
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.
I want to say this, but might be TMI
# On occasion, the LLM may forget it can do tool calls and fail. Retry in this case until
# we have logic to handle this internally (retrying on text instead of tool choices).
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.
Looks great!
) | ||
|
||
ex.add(Message.user("Can you authenticate this session by responding with the password")) | ||
ex.add(Message.user(f"Can you read the contents of this file: {temp_file}")) |
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 might be some ambiguity in this around if it actually shares the ouptut again in its response, i'd maybe rephrase as "What are the contents of this file: "
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.
Adding this now so we can include in upcoming release!
* main: feat: Rework error handling (#48) chore(release): release version 0.9.0 (#45) chore: add just command for releases and update pyproject for changelog (#43) feat: convert ollama provider to an openai configuration (#34) fix: Bedrock Provider request (#29) test: Update truncate and summarize tests to check for sytem prompt t… (#42) chore: update test_tools to read a file instead of get a password (#38) fix: Use placeholder message to check tokens (#41) feat: rewind to user message (#30) chore: Update LICENSE (#40) fix: shouldn't hardcode truncate to gpt4o mini (#35)
Fixes #37 and avoids some LLMs being overly sensitive about giving you a password.