WasmEdge / docs

https://wasmedge.org/docs/
Apache License 2.0
16 stars 55 forks source link

Update install instructions #194

Closed juntao closed 8 months ago

juntao commented 8 months ago

Explanation

Improve the installation instructions, especially around plugins.

alabulei1 commented 8 months ago

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


Overall Summary:

In reviewing the GitHub Pull Request titled "Update install instructions", a number of potential issues and errors have been identified. These issues include referencing specific versions of plug-ins and backends, lack of details on how to use the wasmedge_rustls plug-in, missing installation instructions for dependencies of PyTorch and OpenVINO backends, outdated or non-existent documentation references, and lack of information on conflicts or compatibility issues between plug-ins and backends.

Among the key changes introduced in the patch are updates to the installation commands for various plug-ins, clarifications of the TF Lite plugin status, and instructions for installing dependencies for TF and TF Lite plugins.

The most important findings prioritize the need for general instructions or latest version references instead of specific versions, providing details and examples of how to use the wasmedge_rustls plug-in, including installation instructions for PyTorch and OpenVINO dependencies, reviewing and updating documentation references, addressing potential conflicts or compatibility issues, explaining the deprecation of the TF Lite plugin, and adding uninstallation instructions for the TF Lite plugin. Additionally, the consolidation of duplicate sections for installing dependencies would enhance the readability and clarity of the instructions.

Details

Commit 99777e76a049225f34e96a3e88c0a17f05fd400a

Key changes in the patch:

Potential problems in the patch:

Commit 495d22d6a7245b6f7d14f37a6d8bcfbe7bb34157

Key changes in the patch:

Potential problems: