Skip to content

Add a way to pass a transform matrix along with Stream::setAcquiredImage #8496

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

Merged
merged 32 commits into from
Mar 10, 2025

Conversation

ajmalk
Copy link
Contributor

@ajmalk ajmalk commented Mar 6, 2025

Followup to #8493

Streams in OpenGL are either backed by SurfaceTexture (Native in filament) or ImageReader (Acquired in filament). SurfaceTextures have a transform attached to the image that's updated every frame but have to be read from the GL thread. The ImageReader path has the same transform but is read from the main thread. The transform in the SurfaceTexture path can be optionally requested in the shader (see #8490). This PR adds a way for the ImageReader path to also update this transform.

@ajmalk ajmalk requested a review from pixelflinger March 6, 2025 17:12
sidreesshah and others added 16 commits March 6, 2025 20:16
Replace cmakelist content with few set and includes
Add the patch file to convert dawn shared to static lib

Thanks Robbie Bridgewater for helping out with this implementation
Introduce FILAMENT_SUPPORTS_WEBGPU flag to be used for webgpu support.
Add -W option to  build.sh for webgpu support compilation from build script.

Thanks Andy Hovingh for helping out with this implementation
Introduce Enums to support WebGPU.
Start to use header files from dawn.
Updated the `terminate` function to correct resource cleanup. Changed `mUsedStages` to `mUsedImages` and `mFreeStages` to `mFreeImages` to ensure proper memory management during termination.
* Enable fgviewer for the users

* Fix incorrect position

* Update

* Support remote debug server

* Enable fgviewer on android

* Link and include fgviewer_resources

* Adjust js to show untitled view
@ajmalk ajmalk requested a review from pixelflinger March 7, 2025 23:42
Copy link
Collaborator

@pixelflinger pixelflinger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We're only left with minor style issues. Implementation looks good.

@pixelflinger pixelflinger requested a review from poweifeng March 10, 2025 04:30
@pixelflinger pixelflinger added the internal Issue/PR does not affect clients label Mar 10, 2025
@ajmalk ajmalk requested a review from pixelflinger March 10, 2025 15:17
@pixelflinger pixelflinger merged commit b9a39bc into google:main Mar 10, 2025
1 check passed
z3moon pushed a commit that referenced this pull request Mar 10, 2025
…age (#8496)

BUGS=[399959254]
---------

Co-authored-by: Syed Idris Shah <[email protected]>
Co-authored-by: Haneul Kim <[email protected]>
Co-authored-by: Powei Feng <[email protected]>
Co-authored-by: Doris Wu <[email protected]>
Co-authored-by: Mathias Agopian <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internal Issue/PR does not affect clients
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants