NovelRT icon indicating copy to clipboard operation
NovelRT copied to clipboard

GraphicsPipelineBlendFactor needs reworking.

Open RubyNova opened this issue 3 years ago • 1 comments

This issue is an offshoot of imperfections from #376 .

I have introduced a new enum for handling blend modes, however it is just a direct copy of Vulkan currently. We need to make a superset of the different blend modes between D3D12 and Vulkan, and probably metal as well, however we mostly care about Vulkan and D3D12.

The D3D documentation and the Vulkan documentation will list the relevant enums that need to be compared for the creation of our superset enum. In the event of the two stacks using different names for the same blend factor, please do call it our in the PR so maintainers such as myself and @tannergooding can review the chosen name. Alternatively, feel free to ask about the name either on this issue, on your PR, or on our discord.

RubyNova avatar Jan 24 '22 23:01 RubyNova

I have created the PR for this, there was no need to create a superset as both D3D12 and Vulkan support the same blend operations according to their docs. D3D12_BLEND docs VkBlendFactor docs MtmlBlendFactor docs

RyadaProductions avatar Mar 19 '23 00:03 RyadaProductions