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.

Sign up for GitHub

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

Jump to bottom

Expose depth/stencil state. #17

Closed
crud89 opened this issue May 4, 2021 · 0 comments · Fixed by #36
Closed

Expose depth/stencil state. #17

crud89 opened this issue May 4, 2021 · 0 comments · Fixed by #36
Assignees
@crud89
Labels
DX12 ❎ The issue involves the DX12 backend. Feature Request New feature or request. Priority: Medium A issue with normal priority. Requirement An implementation is required before the next release. Vulkan 🌋 The issue involves the Vulkan backend.
Milestone
Alpha #02

Comments

@crud89
Copy link
Owner

crud89 commented May 4, 2021

Currently the depth/stencil state can only be enabled or disabled. This should be enhanced by exposing the depth and stencil states on a pipeline.

The depth state should expose:

  • Bias.
  • Compare operation. †
  • Test enable/disable. †
  • Write enable/disable. †
  • Depth boundaries.
  • Boundaries test enable/disable. †

The stencil state should expose:

  • Test enable/disable. †
  • Compare mask.
  • Write mask.
  • Reference value.

† Part of the static state, i.e. if modified, the pipeline needs to be recreated.
@crud89 crud89 added 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 May 4, 2021
@crud89 crud89 added this to the Alpha #01 milestone May 4, 2021
@crud89 crud89 self-assigned this May 4, 2021
@crud89 crud89 added the Priority: Medium A issue with normal priority. label May 4, 2021
@crud89 crud89 modified the milestones: Alpha #01, Alpha #02 Jun 18, 2021
@crud89 crud89 mentioned this issue Jun 23, 2021
Open
crud89 added a commit that referenced this issue Jun 23, 2021
@crud89
Expose depth/stencil state.
515a0f6 
Fixes issue #17.
@crud89 crud89 mentioned this issue Jun 23, 2021
Merged
4 tasks
crud89 added a commit that referenced this issue Jun 24, 2021
@crud89
Add method to set the stencil reference.
24aed2d 
Relates to issue #17.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@crud89 crud89

Labels
DX12 ❎ The issue involves the DX12 backend. Feature Request New feature or request. Priority: Medium A issue with normal priority. Requirement An implementation is required before the next release. Vulkan 🌋 The issue involves the Vulkan backend.
Projects
LiteFX
Archived in project
Milestone
Alpha #02
Development

Successfully merging a pull request may close this issue.

Expose missing rasterizer states. crud89/LiteFX
1 participant
@crud89