docs: split a733 and t527 command in Allwinner NPU docs #1016
Conversation
![windsurf-bot[bot]](https://avatars.githubusercontent.com/in/1066231?s=60&v=4){kind=small}
There was a problem hiding this comment.
Other comments (1)
- docs/common/ai/_cubie_lenet.mdx (44-44) The new command `cp ../scripts/* .` is added without explanation. Consider adding a brief comment explaining what these scripts are and why they need to be copied.
💡 To request another review, post a new comment with "/windsurf-review".
docs/common/ai/_cubie_vpm_run.mdx
Outdated
|
|
||
| ```bash | ||
| cd INSTALL_PREFIX/etc/npu/vpm_ru | ||
| cd $INSTALL_PREFIX/etc/npu/vpm_ru |
There was a problem hiding this comment.
There's a typo in the directory path. It should be 'vpm_run' instead of 'vpm_ru'.
| export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:/home/rock/ai-sdk/viplite-tina/lib/aarch64-none-linux-gnu/v2.0 # NPU_SW_VERSION | ||
| ``` | ||
|
|
||
| </NewCodeBlock> | ||
|
|
||
| </TabItem> | ||
|
|
||
| <TabItem value="T527"> | ||
|
|
||
| <NewCodeBlock tip="Device" type="device"> | ||
| ```bash | ||
| export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:/home/rock/ai-sdk/viplite-tina/lib/aarch64-none-linux-gnu/v1.13 # NPU_SW_VERSION | ||
| ``` | ||
| </NewCodeBlock> | ||
|
|
||
| </TabItem> | ||
|
|
||
| </Tabs> | ||
|
|
||
| :::tip | ||
| 指定 NPU_SW_VERSION， A733 选择 `v2.0`, T527 选择 `v1.13`, NPU 信息对照请参考 [NPU 版本对照表](./cubie_acuity_usage#npu-版本对照表) |
There was a problem hiding this comment.
There's an inconsistency in the NPU version information. In the A733 tab (line 327), the export command shows v2.0 as the NPU_SW_VERSION, but the tip below (line 347) also mentions v2.0 for A733. This matches, but it's inconsistent with the earlier section where A733 uses v3 for NPU_VERSION (line 41). Please verify if these should be the same version or if NPU_VERSION and NPU_SW_VERSION are intentionally different.
There was a problem hiding this comment.
NPU_SW_VERSION is difference with NPU_VERSION
|
|
||
| :::tip | ||
| 指定 NPU_VERSION， A733 指定 `v3`, T527 指定 `v1.13`。 信息对照请参考 [NPU 版本对照表](#npu-版本对照表) | ||
| 指定 NPU_VERSION， A733 指定 `v3`, T527 指定 `v2`。 信息对照请参考 [NPU 版本对照表](#npu-版本对照表) |
There was a problem hiding this comment.
There's an inconsistency between the NPU version table and the tip section. In the table, T527 is listed with NPU software version v1.13, but in the tip section it says to specify v2 for T527. These should be consistent to avoid confusion for users.
There was a problem hiding this comment.
NPU_SW_VERSION is difference with NPU_VERSION
Signed-off-by: ZIFENG278 <[email protected]>
3 participants
docs: split a733 and t527 command in Allwinner NPU docs