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

Improve storage buffers. #51

Closed
crud89 opened this issue Jul 7, 2021 · 0 comments · Fixed by #46
Closed

Improve storage buffers. #51

crud89 opened this issue Jul 7, 2021 · 0 comments · Fixed by #46
Assignees
Labels
DX12 ❎ The issue involves the DX12 backend. Feature Request New feature or request. Priority: High A high priority issue. Requirement An implementation is required before the next release. Vulkan 🌋 The issue involves the Vulkan backend.
Milestone

Comments

@crud89
Copy link
Owner

crud89 commented Jul 7, 2021

Currently buffers cannot be written to easily. The Vulkan storage buffer model does not map well to the DirectX UAV model. We should drop storage buffers as a form of "dynamically sized" constant buffers and make textures and buffers optionally writable instead.

@crud89 crud89 added Priority: High A high priority issue. Feature Request New feature or request. Requirement An implementation is required before the next release. Status: In Progress Vulkan 🌋 The issue involves the Vulkan backend. DX12 ❎ The issue involves the DX12 backend. labels Jul 7, 2021
@crud89 crud89 added this to the Alpha #02 milestone Jul 7, 2021
@crud89 crud89 self-assigned this Jul 7, 2021
@crud89 crud89 linked a pull request Jul 7, 2021 that will close this issue
3 tasks
@crud89 crud89 mentioned this issue Jul 20, 2021
3 tasks
crud89 added a commit that referenced this issue Jul 20, 2021
This adds the definitions required for issue #51.
crud89 added a commit that referenced this issue Jul 20, 2021
…for Vulkan back-end.

Relates to #52.
This also implements #51 for Vulkan, though generic buffer writes still need to be tested.
crud89 added a commit that referenced this issue Jul 23, 2021
This should implement #51 for the DirectX 12 back-end.
crud89 added a commit that referenced this issue Jul 23, 2021
This should properly implement #51 for the Vulkan backend.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DX12 ❎ The issue involves the DX12 backend. Feature Request New feature or request. Priority: High A high priority issue. Requirement An implementation is required before the next release. Vulkan 🌋 The issue involves the Vulkan backend.
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant