Update ascend quick install documentation with dynamic configuration#107
Open
hipudding wants to merge 7 commits intoAscend:mainfrom
Open
Update ascend quick install documentation with dynamic configuration#107hipudding wants to merge 7 commits intoAscend:mainfrom
hipudding wants to merge 7 commits intoAscend:mainfrom
Conversation
- Add fetch_ascend_data.py script to fetch latest data from hiascend.com - Add ascend_config.json with version, product series, CPU architectures, OS, and install methods - Refactor ascend_actions.js to load from config and dynamically generate UI - Update quick_install.rst to use dynamic selection - Update Makefile to fetch ascend data on each build - Fix conf.py to handle missing torch_npu gracefully The dynamic selection now supports: - Version selection (8.5.0, 8.5.0.alpha002) - Product series selection - CPU architecture selection - Operating system selection - Install method selection - Dynamic display of installation steps and commands
CLA Signature Passhipudding, thanks for your pull request. All authors of the commits have signed the CLA. 👍 |
- Use CPU core count for worker threads instead of fixed 10 - Replace absolute output path with relative path - Implement task queue pattern for parallel API requests - Delete runtime-generated ascend_config.json from repo
CLA Signature Passhipudding, thanks for your pull request. All authors of the commits have signed the CLA. 👍 |
- Fix shell syntax error in Makefile (removed stray 2. and .) - Restrict torch_npu to Linux platform only (not available on macOS) - Use venv for building
- Use directory targets for submodules to skip if already exists - Make copy-docs depend on submodule directories - Remove clean-submodules call to avoid re-initializing - Use Make's built-in target mechanism instead of shell checks
CLA Signature Passhipudding, thanks for your pull request. All authors of the commits have signed the CLA. 👍 |
zheliuyu
reviewed
Mar 4, 2026
| cpu_arch_id = task['cpu_arch_id'] | ||
| os_id = task['os_id'] | ||
| install_methods = get_install_methods(version_id, hardware_id, cpu_arch_id, os_id) | ||
| print(f" [{threading.current_thread().name}] Found {len(install_methods)} install methods for {os_id}") |
Collaborator
There was a problem hiding this comment.
The meaning of this log message seems a bit unclear. I think we should either clarify what's happening here or just remove it — either is fine.
| @git submodule update --init --remote | ||
|
|
||
| # Copy documentation from submodules | ||
| copy-docs: _repos/verl _repos/VeOmni _repos/LLaMA-Factory _repos/ms-swift |
Collaborator
There was a problem hiding this comment.
Please rebase this part of the code so we can verify the documentation builds successfully during review.
Collaborator
There was a problem hiding this comment.
Additionally, should we also provide a jump link for "More Versions" so that users can access other CANN versions?
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Dynamic ascend version configuration and build fixes
Changes