deepmodeling / deepmodeling/deepmd-kit

ci: build and test Paddle custom operator with CUDA

Open
#5,952 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Python
Stars
2k
Forks
649
Avg merge
6d 18h
Merged PRs (30d)
15

Description

The Paddle border-op CPU-branch regression (source/tests/pd/test_border_op.py) cannot fail in any current CI pipeline:

- `.github/workflows/test_cuda.yml` sets `DP_ENABLE_PADDLE: "0"` at workflow level, so both GPU jobs build without Paddle.
- `.github/workflows/test_python.yml` installs Paddle from the **cpu** nightly index.
- `copy_local_tensor_data` in source/op/pd/comm.cc is compiled only under `GOOGLE_CUDA`/`TENSORFLOW_USE_ROCM`.

Net effect: no configuration builds Paddle with CUDA, so the operator path touched by #5832 is neither compiled nor executed in CI. A regression to an unconditional `gpuMemcpy` would go undetected.

Proposal: add a CI configuration (or a manual/GPU workflow) that builds the Paddle custom operator with CUDA so the `test_border_op_self_copy_uses_cpu_place` CPU-branch test and the rest of source/op/pd/ get real coverage.

Related: #5832

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.