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

Unity integration improvements #22

Open
wants to merge 175 commits into
base: r1.30
Choose a base branch
from
Open

Unity integration improvements #22

wants to merge 175 commits into from

Conversation

wenzhang-unity
Copy link

This PR reworks many features of the RenderStream plugin for Unity. A highlight of the improvements:

  • Integration with Cluster Display package to allow game state synchronization between leader and followers. Awaiting public release https://github.com/Unity-Technologies/ClusterDisplay
  • Camera capture abstraction layer
  • DX12 support (input and output textures)
  • IL2CPP support
  • Debug presenter (allows viewing the inputs or outputs on the render node's desktop).
  • Fixed texture format for uncompressed formats (color space bug)

wenzhang-unity and others added 30 commits October 27, 2022 16:56
@wenzhang-unity
First pass Cluster Display integration
8cdee1f
@wenzhang-unity
DRY up FrameData processing
0dcf67c
@wenzhang-unity
Refactor: Split up DisguiseCameraCapture.cs
a24d35f
@wenzhang-unity
Eliminate GC alloc in hot code.
ca5ab1d
@wenzhang-unity
Reorganize files
f0a77c7
@wenzhang-unity
Bug fixes. Further reduce GC allocs
878d7ad
@wenzhang-unity
More robust scene post-processing
b69de25
@wenzhang-unity
Cleanup. Move FrameSender to its own file
695bf40
@wenzhang-unity
Handle RepeatersDelayedOneFrame case
87e8c8c
@wenzhang-unity
Comments, touchups.
17cfe60
@wenzhang-unity
Address some PR comments
b827e65
@wenzhang-unity
Rework DisguiseRenderStream into non-static class
4b6aed4 
Fix up some initialization logic.

Refactoring: DRY, cleaner cluster handling logic
@wenzhang-unity
Add script to add/set node info in registry
3ac5397
@wenzhang-unity
Rename file to match class
d556a89
@wenzhang-unity
Merge pull request #1 from wenzhang-unity/unity/cluster-display-integ…
0eaddf4 
…ration

Cluster Display integration
@mark-beiline
Native plugin for access to DX12 device and command queue
3f72162
@mark-beiline
Progress
9820e5b
@mark-beiline
Named texture
964e299
@mark-beiline
Cleanup native code
8d030d4
@mark-beiline
C# query shared heap flag
a45d4fd
@FrederickUnity
No need to open the registry key for writting
21a8f3d
@FrederickUnity
Merge pull request #3 from wenzhang-unity/frederick/fix-opensubkey
ccff67e 
No need to open the registry key for writting
@mark-beiline
Applied feedback
c9d3bdf
@mark-beiline
Converted tabs to spaces
8d32fdc
@mark-beiline
C# helper class
a6d63cc
@mark-beiline
Testing confirmed "MarshalAs" is necessary
f5e98de
@mark-beiline
Fixed error message formatting
28a6cb8
@mark-beiline
Merge pull request #2 from wenzhang-unity/mark/dx12
f4e11c3 
DX12 support
@wenzhang-unity
Work with CD's new initialization mechanism
51980fd
@wenzhang-unity
Merge pull request #4 from wenzhang-unity/wen/rework-cluster-init
1965d39 
Work with CD's new initialization mechanism
mark-beiline and others added 30 commits March 30, 2023 13:48
@mark-beiline
Renamed DisguisePresenter -> UnityDebugWindowPresenter
9b30f6f
@mark-beiline
Merge pull request #22 from wenzhang-unity/mark/present-to-screen-v3
59fd78a 
Zero-click presenter with texture selection dropdown
@mark-beiline
Merge pull request #15 from wenzhang-unity/mark/separate-assemblies
aae2a7a 
Separate assembly for pipeline abstraction
@mark-beiline
Merge pull request #14 from wenzhang-unity/mark/enter-play-mode
254a6da 
Cleanup for Editor Play Mode
@mark-beiline
First pass
e9031e4
@mark-beiline
Removed unused code
5da05d7
@mark-beiline
Merge pull request #23 from wenzhang-unity/mark/project-settings-ui
0f601af 
Project settings UI
@mark-beiline
First pass
c68a8d5
@mark-beiline
First pass
7e011e7
@mark-beiline
Update setup instructions in README
02f136f
@mark-beiline
Added link to the Package Manager Window manual
8c51cae
@mark-beiline
Merge pull request #25 from wenzhang-unity/mark/gfx-callbacks-refactor
482b791 
Graphics callbacks simplification
@mark-beiline
Update version
cb14049
@mark-beiline
Merge branch 'dev' into mark/package-structure
8e9ff7c
@mark-beiline
Fix inconsistent case warning on readme.md.meta
2391ef9
@mark-beiline
Fix asmdefs
3a97065
@mark-beiline
Trailing character
b1986c9
@mark-beiline
Merge pull request #28 from wenzhang-unity/mark/fixes
d2ac69b 
Assorted bugfixes
@mark-beiline
Merge branch 'dev' into mark/package-structure
4e0c226
@mark-beiline
Merge pull request #26 from wenzhang-unity/mark/package-structure
274318b 
Package structure
@FrederickUnity
Fix update of remote texture with cluster sync
d9adccf
@FrederickUnity
Merge pull request #30 from wenzhang-unity/frederick/fix-remote-textu…
348e4e3 
…re-update-with-cluster-sync

Fix update of remote texture with cluster sync
@mark-beiline
Fixed
706bb5e
@mark-beiline
Simplified
c9ff00e
@wenzhang-unity
Add cluster setup instructions (#32)
dcf0542 
* Add cluster setup instructions

* Wording
@mark-beiline
Works
027f50b
@mark-beiline
Same but simpler
026dcc0
@mark-beiline
Merge pull request #31 from wenzhang-unity/mark/fix-uncompress-v2
727a11b 
Fix texture format for uncompressed RenderStream formats
@mark-beiline
Added struct
370575c
@mark-beiline
Merge pull request #33 from wenzhang-unity/mark/presenter-indexing-fix
e7e966b 
Indexing fix for debug presenter
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@wenzhang-unity @mark-beiline @FrederickUnity