Skip to content

Conversation

@Potabk
Copy link
Collaborator

@Potabk Potabk commented Jun 17, 2025

What this PR does / why we need it?

  • For npu environment, we should use PYTORCH_NPU_ALLOC_CONF rather than PYTORCH_CUDA_ALLOC_CONF
  • Add PIP_EXTRA_INDEX_URL to make nightly_benchmarks happy

Does this PR introduce any user-facing change?

How was this patch tested?

Potabk added 3 commits June 17, 2025 14:14
Signed-off-by: wangli <wangli858794774@gmail.com>
Signed-off-by: wangli <wangli858794774@gmail.com>
Signed-off-by: wangli <wangli858794774@gmail.com>
@Potabk
Copy link
Collaborator Author

Potabk commented Jun 17, 2025

@wangxiyuan
Copy link
Collaborator

It's good to add ut as well. It can be done in the future with sleep mode unit test together

@wangxiyuan wangxiyuan merged commit f802994 into vllm-project:main Jun 17, 2025
20 checks passed
@Yikun Yikun added this to the v0.9.1 milestone Jun 23, 2025
shiyuan680 pushed a commit to raindaywhu/vllm-ascend that referenced this pull request Jul 7, 2025
…m-project#1252)

### What this PR does / why we need it?
- For npu environment, we should use `PYTORCH_NPU_ALLOC_CONF ` rather
than `PYTORCH_CUDA_ALLOC_CONF`
- Add `PIP_EXTRA_INDEX_URL` to make nightly_benchmarks happy


---------

Signed-off-by: wangli <wangli858794774@gmail.com>
chopper0126 pushed a commit to chopper0126/vllm-ascend that referenced this pull request Oct 16, 2025
…m-project#1252)

### What this PR does / why we need it?
- For npu environment, we should use `PYTORCH_NPU_ALLOC_CONF ` rather
than `PYTORCH_CUDA_ALLOC_CONF`
- Add `PIP_EXTRA_INDEX_URL` to make nightly_benchmarks happy


---------

Signed-off-by: wangli <wangli858794774@gmail.com>
Angazenn pushed a commit to Angazenn/vllm-ascend that referenced this pull request Oct 21, 2025
…m-project#1252)

### What this PR does / why we need it?
- For npu environment, we should use `PYTORCH_NPU_ALLOC_CONF ` rather
than `PYTORCH_CUDA_ALLOC_CONF`
- Add `PIP_EXTRA_INDEX_URL` to make nightly_benchmarks happy


---------

Signed-off-by: wangli <wangli858794774@gmail.com>
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.

3 participants