Skip to content

[DLPack] Minor change on DLConvertor naming#166325

Closed
Kathryn-cat wants to merge 1 commit intopytorch:mainfrom
Kathryn-cat:kathy/dlpack-api-minor-change
Closed

[DLPack] Minor change on DLConvertor naming#166325
Kathryn-cat wants to merge 1 commit intopytorch:mainfrom
Kathryn-cat:kathy/dlpack-api-minor-change

Conversation

@Kathryn-cat
Copy link
Contributor

@Kathryn-cat Kathryn-cat commented Oct 27, 2025

This PR introduces minor changes to DLConvertor function and naming.

  • getATenDevice is changed to dlDeviceToTorchDevice and removed being exposed as static function, in order to be symmetric with torchDeviceToDLDevice.
  • toDLPackNonOwning is introduced to convert torch tensor to a non-owning DLPack tensor.

@pytorch-bot
Copy link

pytorch-bot bot commented Oct 27, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/166325

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit 062535c with merge base 058782c (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@Kathryn-cat Kathryn-cat marked this pull request as ready for review October 27, 2025 18:57
@Kathryn-cat
Copy link
Contributor Author

@pytorchbot label "topic: not user facing" label "module: dlpack"

@pytorch-bot
Copy link

pytorch-bot bot commented Oct 27, 2025

Didn't find following labels among repository labels: label

@soulitzer soulitzer requested review from albanD and malfet October 28, 2025 15:39
@soulitzer soulitzer added the triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module label Oct 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module: dlpack open source topic: not user facing topic category triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants