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

Device States and Backend Switching #66

Merged
merged 122 commits into from
Jul 28, 2022
Merged

Device States and Backend Switching #66

merged 122 commits into from
Jul 28, 2022

Conversation

crud89
Copy link
Owner

@crud89 crud89 commented Jul 27, 2022 •
edited
Loading

Describe the pull request

This PR introduces several API improvements:

  • DeviceState allows to let the device manage resources, namely render passes, pipelines, buffers, images, samplers and descriptor sets.
  • Apps can now be configured with multiple backends per backend type. Using device states help to manage resources over multiple backends.
  • The builder interface has been improved and is now less intrusive, as well as more compatible. This also helps when creating states for multiple backends from a single (template) method.

Device state resources also can be assigned a name now, which is also a preparation for #15.

Related issues

ToDo

Currently, it is not possible to switch from DX12 back to Vulkan, since D3D12 only supports flip-model based swap chains and there is no way to reset the window afterwards. To fix this, the Vulkan swap chain should interop the DXGI swap chain if the DX12 backend is available. This should also improve performance for Vulkan, since it does not rely on legacy GDI bitblt anymore.

Also the readme and documentation should be adapted before merging.

@crud89
Fix documentation comments.
21bdc7d
@crud89
Ensure that ANSI color codes are properly displayed in the Windows te…
99f4eac 
…rminal.
@crud89
Replace codecvt on Win32.
505130c
@crud89
Remove codecvt deprecation warning silencer.
d91c0cf
@crud89
Use string views for widen and narrow.
4e85150
@crud89
Switch to string views for logging and exceptions.
c3d2123
@crud89
Add name member to backend interface.
fe02f52 
Also rename `getType` member.
@crud89
Refactored properties on app base class.
ccc61e6
@crud89
Adjust name lookup for backends.
22d04f5
@crud89
Remove name member from backend interface.
9c72a02
@crud89
Initialize instead of run app during build.
53ab85e
@crud89
Move backend state to interface.
a699b95
@crud89
Store backend initializer in app instance.
3e8e396
@crud89
Allow overwriting backend initializers.
39588af
@crud89
Move surface ownership to device.
f5b8234
@crud89
Add (de-)activation methods to backend interface.
4aa3bcc
@crud89
Add state resource base class.
d645967
@crud89
Merge current main.
696bc0a
@crud89
Move backend conversion helpers into their own namespaces.
1069552
@crud89
Implement backend state.
6b92075
@crud89
Implement backend start and stop callbacks.
0a8e892
@crud89
Fix condition in pimpl destructor.
78b6b5b
@crud89
Implement device management on backends.
0130d44
@crud89
Unrestrict device management on backends.
f0e16ee
@crud89
Introduce flat interfaces for backend and device.
e708207
@crud89
Add nodiscard attribute to all final builder calls.
c3fadc6
@crud89
Add name parameter for render pass builder.
5b1eeb9
@crud89
Isolate swap chain interface.
2eb5633
@crud89
Fix warning about improper negation.
c4de8e3
@crud89
Isolate graphics factory interface.
ccb9330
crud89 added 26 commits July 28, 2022 15:18
@crud89
Port render passes sample.
53ed673
@crud89
Add readme for sample directory.
fd5bf52
@crud89
Port push constants sample.
4a51164
@crud89
Update example in readme.
a126673
@crud89
Rename push constants range builder interface.
21f3eb6
@crud89
Add switching backends to features list.
be21a2f
@crud89
Port uniform buffer arrays sample.
792ad18
@crud89
Fix push constants definition.
49a4e1f
@crud89
Fix min/max LOD for sampler.
0071955
@crud89
Add device interface method for creating barriers.
171b18d
@crud89
Only add UAV barriers for writable resources.
c533118
@crud89
Port textures sample.
71622e3
@crud89
Enable tessellation and geometry shader features.
288f300
@crud89
Set initial texture layout to transfer destination.
f7e81b2
@crud89
Fix validation errors when generating mip maps.
d7fe49e
@crud89
Writable images can also be sampled.
10ec893
@crud89
Re-enable mip-map generation.
1378704
@crud89
Add description for textures sample.
bcdc164
@crud89
Port multi-sampling sample.
01995a2
@crud89
Port multi-threading sample.
6f5d599
@crud89
Remove legacy samples.
c7e9b67
@crud89
Remove launch page.
acb7972
@crud89
Update website.
b3826b0
@crud89
Update quick start guide.
61395da
@crud89
Add new release log.
586e28a
@crud89
Update website.
fb228fd
@crud89 crud89 marked this pull request as ready for review July 28, 2022 18:22
@crud89 crud89 merged commit 849f2ff into main Jul 28, 2022
@crud89 crud89 deleted the backend-states branch July 28, 2022 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@crud89 crud89

Labels
Module: DX12 ❎ The issue involves the DX12 backend. Module: Vulkan 🌋 The issue involves the Vulkan backend. Priority: High A high priority issue. Type: Requirement An implementation is required before the next release.
Projects
LiteFX
Status: v0.3.1
Milestone
Alpha #03
Development

Successfully merging this pull request may close these issues.

Interop DXGI swap chain in Vulkan, if DX12 backend is enabled. Support back-end switching.
1 participant
@crud89