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

Fix the outdated llm document #210

Merged
merged 1 commit into from
Feb 2, 2024
Merged

Fix the outdated llm document #210

merged 1 commit into from
Feb 2, 2024

Conversation

hydai
Copy link
Member

@hydai hydai commented Feb 2, 2024

Explanation

  1. llama-utils is renamed to llamaedge
  2. The links are all broken.

This PR fixed the above issues.

Related issue

Fixed WasmEdge/WasmEdge#3192

What type of PR is this

/kind documentation

Proposed Changes

Make things right.

Signed-off-by: hydai <z54981220@gmail.com>
Copy link
Collaborator

alabulei1 commented Feb 2, 2024

Hello, I am a code review bot on flows.network. Here are my reviews of code commits in this PR.


Commit 37aed9011aa62b626178a6da9e6968550ac5b452

Key changes in the patch:

  • Updated the link to the example project in the introduction.
  • Added a list of supported models with links to their details.
  • Updated the link to the llama-chat.wasm file in the quick start section.
  • Updated the link to the llama2 7b chat model in the curl command.
  • Updated the link to the llamaedge repository in the build and run section.
  • Updated the link to the huggingface model in the table.
  • Updated the link to the main.rs file in the Understand the code section.

Potential problems:

  • The links in the patch may not be accurate or up to date. It's important to verify that the links are correct and accessible.
  • The changes may introduce new issues or break existing functionality. A thorough testing of the updated llm document is required.

Copy link
Member

@juntao juntao left a comment

Choose a reason for hiding this comment

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

Thanks!

@juntao juntao merged commit cf59767 into main Feb 2, 2024
6 checks passed
@juntao juntao deleted the hydai/fix_llm branch February 2, 2024 20:44
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.

bug: --nn-preload option doesn't work with the latest version of WasmEdge (v0.13.5)
3 participants