Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DOC]: DeviceSelect::If upper limit of temp_storage_bytes? #1148

Open
1 task done
isovic opened this issue Nov 24, 2023 · 2 comments
Open
1 task done

[DOC]: DeviceSelect::If upper limit of temp_storage_bytes? #1148

isovic opened this issue Nov 24, 2023 · 2 comments
Assignees
Labels
doc Documentation-related items. needs triage Issues that require the team's attention

Comments

@isovic
Copy link

isovic commented Nov 24, 2023

Is this a duplicate?

Is this for new documentation, or an update to existing docs?

Update

Describe the incorrect/future/missing documentation

The DeviceSelect module documentation lacks information on an estimated (or better yet, upper bound) memory required for the temporary storage. Which parameters does it scale with (e.g the length of the input, number of threads, or is it fixed size)?

For example, the DeviceRadixSort docs provide information as the following:

This operation requires an allocation of temporary device storage that is O(N+P), where N is the length of the input and P is the number of streaming multiprocessors on the device. For sorting using only O(P) temporary storage, see the sorting interface using DoubleBuffer wrappers below.

However, for DeviceSelect::If I cannot tell if the temp_storage_bytes will even fit my memory for giant inputs or not, and I cannot batch my array beforehand because I don't know how temp storage scales.

Before the docs are officially updated, can you please provide some information about the estimate here?

Thank you,
Best regards,
Ivan.

If this is a correction, please provide a link to the incorrect documentation. If this is a new documentation request, please link to where you have looked.

No response

@isovic isovic added the doc Documentation-related items. label Nov 24, 2023
@github-project-automation github-project-automation bot moved this to Todo in CCCL Nov 24, 2023
@github-actions github-actions bot added the needs triage Issues that require the team's attention label Nov 24, 2023
Copy link
Contributor

Hi @isovic!

Thanks for submitting this issue - the CCCL team has been notified and we'll get back to you as soon as we can!
In the mean time, feel free to add any relevant information to this issue.

@miscco miscco assigned gevtushenko and unassigned miscco Nov 24, 2023
@jrhemstad
Copy link
Collaborator

At present, we don't document anything other than the temporary memory required is implementation defined. We've had similar requests recently, so we will likely try to do something more.

The problem is that providing an exact number or even robust estimate ties our hands on ever changing or improving the implementation in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Documentation-related items. needs triage Issues that require the team's attention
Projects
Status: Todo
Development

No branches or pull requests

4 participants