Support for multicast/unicast?

Open
#191 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
5/5
Estimated time
Over a week
Newbie friendliness
30/100
Issue type
Feature
Clarity
Needs clarification
Activity status
Active
Tech stack
cpp
Domain
api

Research direction

No files or tests are named. Start by reviewing the current device enum, tensor struct, and flag definitions, then clarify whether multicast requires a new device and whether unicast belongs in optional metadata; done when the representation and compatibility scope are agreed.

Written by the indexing model from the issue text.

Description

This came up in a discussion whether one could support multicast/unicast in DLPack.

I don't think this is used currently by things like CuPy/torch but other users seem to have the need to share this information.
As I am not very familiar with multicast/unicast, I think a multicast array would actually be it's own device, while a unicast could be a normal CUDA allocation, but unicast is still good information.
If unicast here is sufficient as optional metadata, then it could be a flag. Although I guess it might get us into a place where we may want device specific flags.

If there is a use-case for adding devices, that seems just very cheap to me (a simple enum entry after all), flags also are, although if we want to do device-specific flags/metadata we may want to extend the struct (or reserve some of the flag space).

Dominant language
C++
Stars
1.3k
Forks
168
Avg merge
1m
Merged PRs (30d)
2

Getting set up

This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from dmlc/dlpack

All issues in dmlc/dlpack

Similar issues

More C++ issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.