Repository navigation
Custom Materials with Atomic Counter #7030
Description
What problem does this solve or what need does it fill?
After #6988 I've questioned myself if it would be also possible to make custom materials use atomic counter for their AsBindGroup, instead of TypeUUID.
What solution would you like?
Make the AsBindGroup derive also increasing the atomic counter, so that lines like
bevy/examples/shader/shader_material.rs
Line 57 in b6066c3
Activity
james7132 commentedon Dec 25, 2022
That line is for the TypeUuid derive. It's required to support using it as an asset. The use case is entirely different and definitely needs a stable independent unique ID that is the same across compilations and executions of the same binary. I don't think we can get rid of that without the stable type name as a replacement for TypeUuid.
DasLixou commentedon Dec 26, 2022
Oh I See.. should this be closed?
james7132 commentedon Jan 9, 2023
Probably should. I don't think this issue is actionable. Feel free to reopen if there is something tangible we could do to improve our use of UUIDs here.