langchain-ai / langchainjs

🦜🔗 Build context-aware reasoning applications 🦜🔗
https://js.langchain.com/docs/
MIT License
12.71k stars 2.19k forks source link

docs: Update pdf_qa.ipynb #7137

Closed apuravchauhan closed 1 week ago

apuravchauhan commented 2 weeks ago

Refer the correct variable name

In the tutorial the, llm is instantiated under the variable name model. But while creating the createStuffDocumentsChain in the following steps, the variable name referred is llm.

The issue comes in because the variable llm exists in python documentation and the same is copied here in js documentation Fixes # (issue)

vercel[bot] commented 2 weeks ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
langchainjs-docs ✅ Ready (Inspect) Visit Preview Nov 5, 2024 4:34am
1 Skipped Deployment | Name | Status | Preview | Comments | Updated (UTC) | | :--- | :----- | :------ | :------- | :------ | | **langchainjs-api-refs** | ⬜️ Ignored ([Inspect](https://vercel.com/langchain/langchainjs-api-refs/6U6F1Jwr7VaWR6z3uiaU81UAyoWN)) | | | Nov 5, 2024 4:34am |
jacoblee93 commented 1 week ago

Thank you!