Skip to content

Add DEVICE_FEATURE for depth clamp zero one#1074

Merged
dj2 merged 1 commit into
google:mainfrom
gnl21:gnl-depth-clamp
Jan 9, 2025
Merged

Add DEVICE_FEATURE for depth clamp zero one#1074
dj2 merged 1 commit into
google:mainfrom
gnl21:gnl-depth-clamp

Conversation

@gnl21
Copy link
Copy Markdown
Contributor

@gnl21 gnl21 commented Jan 8, 2025

No description provided.

@gnl21
Copy link
Copy Markdown
Contributor Author

gnl21 commented Jan 8, 2025

I see that the NDK autobuild is failing to find any of the tokens that are used in this change, but the version of the vulkan headers in DEPS does include them. Do you know if there's any special that I should be aware of around that?

@dj2
Copy link
Copy Markdown
Collaborator

dj2 commented Jan 8, 2025

@dneto0 do you remember how the NDK build works? Does it use deps or something out of the NDK? Do we need to update it in some fashion?

@dneto0
Copy link
Copy Markdown
Collaborator

dneto0 commented Jan 8, 2025

I think the in the Android.mk build the vulkan headers are taken from the NDK. The bots use ndk-r25b which I think is acouple of years old.

I'll update the bots.

@dneto0
Copy link
Copy Markdown
Collaborator

dneto0 commented Jan 8, 2025

The NDK r27c includes vulkan.h for 1.3.275, and the needed tokens are in there.

@dneto0
Copy link
Copy Markdown
Collaborator

dneto0 commented Jan 8, 2025

#1076 updates the scripts to use NDK r27c.
Please rebase on top of main

@dj2 dj2 merged commit ffc084a into google:main Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants