-
Notifications
You must be signed in to change notification settings - Fork 59
feat(js): retrieval/integration doc translations #692
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
base: main
Are you sure you want to change the base?
Conversation
Preview ID generated: preview-hunter-1759195386-b13f72a |
66de32b
to
ad39b30
Compare
Preview ID generated: preview-hunter-1759269798-fb6d734 |
Preview ID generated: preview-hunter-1759270187-c8de940 |
print(texts[0]) | ||
``` | ||
```output | ||
page_content='Madam Speaker, Madam Vice President, our First Lady and Second Gentleman. Members of Congress and the Cabinet. Justices of the Supreme Court. My fellow Americans. \n\nLast year COVID-19 kept us apart. This year we are finally together again. \n\nTonight, we meet as Democrats Republicans and Independents. But most importantly as Americans. \n\nWith a duty to one another to the American people to the Constitution. \n\nAnd with an unwavering resolve that freedom will always triumph over tyranny. \n\nSix days ago, Russia’s Vladimir Putin sought to shake the foundations of the free world thinking he could make it bend to his menacing ways. But he badly miscalculated. \n\nHe thought he could roll into Ukraine and the world would roll over. Instead he met a wall of strength he never imagined. \n\nHe met the Ukrainian people. \n\nFrom President Zelenskyy to every Ukrainian, their fearlessness, their courage, their determination, inspires the world.' |
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.
Ideally, we'd have output/examples that are non-political and not American-centric to make our docs more accessible to a wider audience, but I think stateOfTheUnion
was an existing example already. Just flagging - we can update this example 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.
yeah these are brought over
:::js | ||
```typescript | ||
import { createAgent } from "langchain"; | ||
import { createAgent, dynamicSystemPromptMiddleware } from "langchain"; |
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.
@christian-bromann are we removing this prebuilt middleware in favor of modifyRequest?
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.
no, we decided to keep dynamicSystemPromptMiddleware
in JS
No description provided.