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

DOC: <Issue related to docs/integrations/llms/llama_cpp.mdx> #7365

Closed
2 tasks done
Jacky3003 opened this issue Dec 14, 2024 · 0 comments · Fixed by #7426
Closed
2 tasks done

DOC: <Issue related to docs/integrations/llms/llama_cpp.mdx> #7365

Jacky3003 opened this issue Dec 14, 2024 · 0 comments · Fixed by #7426
Assignees
Labels
auto:documentation Changes to documentation and examples, like .md, .rst, .ipynb files. Changes to the docs/ folder

Comments

@Jacky3003
Copy link
Contributor

Checklist

  • I added a very descriptive title to this issue.
  • I included a link to the documentation page I am referring to (if applicable).

Issue with current documentation:

With the new update to node-llama-cpp being bumped up to version 3, the integration docs for its coresponding LLM component in LangChain should use some updates in the numbering and model being used in the examples in order to avoid confusion. The following docs as a result should be updated:

Idea or request for content:

It would be nice for the version numbering to be changed from 2 to 3. For the LLM's documentation specifically, it should be investigated if the guide to installing a llama 2 model (which should now be llama 3 in this instance) should be updated as well. I plan to work on this issue.

@dosubot dosubot bot added the auto:documentation Changes to documentation and examples, like .md, .rst, .ipynb files. Changes to the docs/ folder label Dec 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto:documentation Changes to documentation and examples, like .md, .rst, .ipynb files. Changes to the docs/ folder
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant