Skip to content

Fixup acceleration structure and raytracing flags.#1073

Merged
dj2 merged 3 commits intogoogle:mainfrom
dj2:init
Jan 7, 2025
Merged

Fixup acceleration structure and raytracing flags.#1073
dj2 merged 3 commits intogoogle:mainfrom
dj2:init

Conversation

@dj2
Copy link
Collaborator

@dj2 dj2 commented Jan 7, 2025

When determining if acceleration structures and ray tracing are supported it isn't enough to just check for the device extension. Both of these extensions maybe available, then the features contain a flag which determines if it is implemented. So, if we've got the features, we need to additionally check the flag and reset our internal tracking based on the implementation flag.

Change-Id: I09b04007a47e5b5bec34c7ed910949401ca18a3b

When determining if acceleration structures and ray tracing are
supported it isn't enough to just check for the device extension. Both
of these extensions maybe available, then the features contain a flag
which determines if it is implemented. So, if we've got the features, we
need to additionally check the flag and reset our internal tracking
based on the implementation flag.

Change-Id: I09b04007a47e5b5bec34c7ed910949401ca18a3b
@dj2 dj2 added the bug Something isn't working label Jan 7, 2025
@dj2 dj2 requested a review from dneto0 January 7, 2025 18:23
@dj2 dj2 self-assigned this Jan 7, 2025
Copy link
Collaborator

@dneto0 dneto0 left a comment

Choose a reason for hiding this comment

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

please update the comment.

Change-Id: If1710da425312603b5a793c67ca6afe8d9b467a9
@dj2 dj2 requested a review from dneto0 January 7, 2025 18:53
Change-Id: I5ad8a5ea27ac79700a58c858da35681d3683765e
@dj2 dj2 merged commit f4303cd into google:main Jan 7, 2025
1 check passed
@dj2 dj2 deleted the init branch January 7, 2025 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants