Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
GPU based picking with points #1721
GPU based picking with points #1721
Changes from 20 commits
e03f027
c52f20d
4e99981
6b939be
7f037aa
fe1c13f
ae2efe0
bb78b44
203d87c
87ea6fa
68b786d
1004fc0
f39a8c2
956f4d9
6743985
f20f1c3
7091673
0cae682
3051ce8
9efc76c
27e4ea0
5a2a1bc
256c2a3
546c8bd
6f93fff
6105541
e82766e
39263c9
6f98efb
d237a26
c0601f9
e09dea2
969eadb
58b3c21
9ff255d
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I feel already that we should have a helper layer between the user and
gpu_readback_belt
. If every users need to store the original rectangles so that they can remove the padding, then there should be a helper layer that does all that for usThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah that's what I meant with the comment :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
although, not having the user store the originals is kinda hard I think. I didn't want to do dynamic dispatching on the level of the readbackbelt, i.e. not categorizing all the possible kinds of readbacks on the re_renderer side
there's some other work I have planned to deal with the problem of conjoined buffers. I'll have a look into higher level layers then.