Skip to content

[ci] Adjust ci.yml and __init__.py#66

Merged
tianyi-ge merged 2 commits into
Ascend:masterfrom
dpj135:adjust_ci
Jun 10, 2026
Merged

[ci] Adjust ci.yml and __init__.py#66
tianyi-ge merged 2 commits into
Ascend:masterfrom
dpj135:adjust_ci

Conversation

@dpj135

@dpj135 dpj135 commented Jun 3, 2026

Copy link
Copy Markdown
Collaborator

Main Changes

Signed-off-by: dpj135 <958208521@qq.com>
@ascend-robot

Copy link
Copy Markdown

CLA Signature Pass

dpj135, thanks for your pull request. All authors of the commits have signed the CLA. 👍

Comment thread ray_ascend/__init__.py
Comment on lines +160 to +161
register_hccl_collective_backend()

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Unnecessary changes?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

make register_hccl_tensor_transport raises RuntimeError like register_hccl_collective_backend instead of ImportedError.

Comment thread .github/workflows/ci.yml Outdated
run: |
echo "Running tests for Python ${{ matrix.python-version }}..."
python${{ matrix.python-version }} -m pytest -v tests/
- name: Verify backward compatibility (Ray ${{ matrix.ray-version }})

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

better add this script to tests as a unit test (only run if ray==2.55.0). It's not clear to maintain this script in workflow yaml

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

done

@ascend-robot

Copy link
Copy Markdown

CLA Signature Pass

dpj135, thanks for your pull request. All authors of the commits have signed the CLA. 👍

Signed-off-by: dpj135 <958208521@qq.com>
@ascend-robot

Copy link
Copy Markdown

CLA Signature Pass

dpj135, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@tianyi-ge tianyi-ge merged commit 87c1134 into Ascend:master Jun 10, 2026
9 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants