Skip to content

Implement createReActAgent#169

Merged
William FH (hinthornw) merged 10 commits intomainfrom
wfh/cra
May 19, 2024
Merged

Implement createReActAgent#169
William FH (hinthornw) merged 10 commits intomainfrom
wfh/cra

Conversation

@hinthornw
Copy link
Copy Markdown
Contributor

No description provided.

"@jest/globals": "^29.5.0",
"@langchain/community": "^0.0.43",
"@langchain/openai": "^0.0.23",
"@langchain/openai": "latest",
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

to get the bindTools for testing


const callModel = async (state: AgentState) => {
const { messages } = state;
// TODO: Stream
Copy link
Copy Markdown
Contributor Author

@hinthornw William FH (hinthornw) May 18, 2024

Choose a reason for hiding this comment

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

If we wanted to support token-wise streaming via streamEvents, i think i need to actually stream here and concatenate, though i need to check. been a while since ir eviewed the js chat model implementations

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Because in js we don't auto promote invoke to stream like we do in py?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

afaict


const callModel = async (state: AgentState) => {
const { messages } = state;
// TODO: Stream
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Because in js we don't auto promote invoke to stream like we do in py?

@hinthornw William FH (hinthornw) merged commit 7190937 into main May 19, 2024
@hinthornw William FH (hinthornw) deleted the wfh/cra branch May 19, 2024 04:17
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