Skip to content

Conversation

@yq33victor
Copy link
Collaborator

No description provided.

@yq33victor yq33victor force-pushed the dev/support_offline_multi_xpu3 branch from 5d8db2a to 2c0bc1e Compare October 22, 2025 13:33
@yq33victor yq33victor force-pushed the dev/support_offline_multi_xpu3 branch 2 times, most recently from febeb33 to 2f78aa0 Compare October 22, 2025 14:03
Signed-off-by: pengtao.156 <pengtao.156@jd.com>
@yq33victor yq33victor force-pushed the dev/support_offline_multi_xpu3 branch from 2f78aa0 to 0ac5cf7 Compare October 23, 2025 02:10

std::atomic<bool> done(false);
#if defined(USE_NPU)
xllm::Device device("npu:" + std::to_string(device_idx));
Copy link
Collaborator

Choose a reason for hiding this comment

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

maybe we can add another construction function for class Device:

Device::Device(int32_t device_id){
  std::string device_name = type() + ":" + std::to_string(device_id);
  device_ = torch::Device(device_name);
}

and move #if defined(USE_NPU) before FLAGS_enable_atb_comm_multiprocess = true;

@liutongxuan liutongxuan merged commit 23386ef into jd-opensource:main Oct 24, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants