Skip to content

Conversation

@AnastaZIuk
Copy link
Member

No description provided.

small change to make sure `ditt` doesn't autodelete
…ubresource bitflags to not trigger asserts, update examples_tests submodule
…experimental, no real validation), update examples_tests submodule
…arguments in json payload, CI should pass now
… with GPU converter, encode to EF_A2R10G10B10_UNORM_PACK32, update examples_tests submodule
inline float32_t __call(NBL_CONST_REF_ARG(Accessor) accessor, NBL_CONST_REF_ARG(float32_t2) uv) NBL_CONST_MEMBER_FUNC
{
const float32_t2 halfMinusHalfPixel = float32_t2(0.5f, 0.5f) / (float32_t2(1.f, 1.f) + info.lastTexelRcp);
const float32_t2 ndc = (uv - float32_t2(0.5f, 0.5f)) / halfMinusHalfPixel;

Choose a reason for hiding this comment

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

my bad I made this more complicated than should be, seems there should be two separate procedural textures (one for __call with integer texel coordinates and antother for __call with UV) which inherit a base one (where __call takes NDC) and they should have different infos (lastTexelRcp for first and halfMinusHalfTexel for other)

Now neither is fast.

Choose a reason for hiding this comment

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

althought I guess we never really use this while shading, so no changes needed

Comment on lines +72 to +73
template<typename Accessor>
inline float32_t __evalNDC(NBL_CONST_REF_ARG(Accessor) accessor, NBL_CONST_REF_ARG(float32_t2) ndc) NBL_CONST_MEMBER_FUNC

Choose a reason for hiding this comment

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

NBL_FUNC_REQUIRE that Accessor fists a concept

inline candela_t operator()(NBL_CONST_REF_ARG(accessor_t) accessor, NBL_CONST_REF_ARG(polar_t) polar) NBL_CONST_MEMBER_FUNC
{
assert(polar.theta >= float32_t(0.0) && polar.theta <= numbers::pi<float32_t>);
assert(hlsl::abs(polar.phi) <= numbers::pi<float32_t> * float32_t(2.0));

Choose a reason for hiding this comment

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

huh why do you allow -2*pi < phi < 2*pi ?

Copy link
Member Author

Choose a reason for hiding this comment

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

and that one I probably broke/typoed

@devshgraphicsprogramming devshgraphicsprogramming merged commit 4e3b22f into master Feb 5, 2026
19 of 20 checks passed
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.

2 participants