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

Create Langchain Genie #5

Merged
merged 7 commits into from
Oct 24, 2024
Merged

Create Langchain Genie #5

merged 7 commits into from
Oct 24, 2024

Conversation

prithvikannan
Copy link
Collaborator

@prithvikannan prithvikannan commented Oct 22, 2024

Create a GenieAgent in databricks_langchain

Signed-off-by: Prithvi Kannan <[email protected]>
Signed-off-by: Prithvi Kannan <[email protected]>
@prithvikannan prithvikannan changed the title Langchain genie Create Langchain Genie Oct 22, 2024
Signed-off-by: Prithvi Kannan <[email protected]>
Signed-off-by: Prithvi Kannan <[email protected]>
Signed-off-by: Prithvi Kannan <[email protected]>
Signed-off-by: Prithvi Kannan <[email protected]>
return {"messages": [AIMessage(content="")]}


def GenieAgent(genie_space_id, genie_agent_name="Genie", description=""):
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

adding description to the interface, but TBD on how this will be used.

Copy link

@aravind-segu aravind-segu left a comment

Choose a reason for hiding this comment

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

LGTM


genie = Genie(genie_space_id)

message = f"I will provide you a chat history, where your name is {genie_agent_name}. Please help with the described information in the chat history.\n"

Choose a reason for hiding this comment

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

Question here: Are we expecting users to use the genie agent name in their questions? Will the questions be something like:

"Use Genie to get me my financial records"

Will this also work when users ask "Get my financial records"

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The ideal end state is that the user can just specify "Get my financial records". This name is needed so that in a multi-agent (and in the future multi-user) chat array agents can identify who said what.

@prithvikannan prithvikannan merged commit 1db0425 into main Oct 24, 2024
7 checks passed
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 this pull request may close these issues.

2 participants