Skip to content
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

Sampler name not printed out in an error message #170

Closed
speedym opened this issue Feb 21, 2022 · 4 comments
Closed

Sampler name not printed out in an error message #170

speedym opened this issue Feb 21, 2022 · 4 comments

Comments

@speedym
Copy link

speedym commented Feb 21, 2022

When unpacking the archive, sampler name is not shown in the error message:

Diligent Engine: ERROR in ValidateSamplerDesc() (SamplerBase.cpp, 46): Description of sampler '' is invalid: Subsampled sampler requires SHADING_RATE_CAP_FLAG_SUBSAMPLED_RENDER_TARGET capability
Diligent Engine: ERROR in CreateDeviceObject() (RenderDeviceBase.hpp, 511): Failed to create Sampler object ''
Diligent Engine: ERROR: Debug assertion failed in GetVkSampler(), file PipelineResourceSignatureVkImpl.cpp, line 183:
Debug expression failed:
Ptr

With the following .json contents:

        "ImmutableSamplers": [
            {
                "ShaderStages": [ "PIXEL" ],
                "SamplerOrTextureName": "vrs_texture",
                "Desc": {
                    "MinFilter": "POINT",
                    "MagFilter": "POINT",
                    "MipFilter": "POINT",
                    "MinLOD": 0.0,
                    "MaxLOD": 0.0,
                    "Flags": [ "SUBSAMPLED" ]
                }
            }
        ]
@TheMostDiligent
Copy link
Contributor

Note that SamplerOrTextureName is the name of the sampler or texture in the shader that the sampler object is assigned to.
The name of the sampler object should be defined in the Desc, and it is missing in this case.

@speedym
Copy link
Author

speedym commented Feb 22, 2022

I have just checked and SamplerDesc struct does not contain the Name field. Also, ImmutableSamplerDesc struct does not contain the Name field as well. So it seems we can't explicitly name samplers?

@MikhailGorobets
Copy link
Contributor

@speedym
Copy link
Author

speedym commented Feb 22, 2022

Ah, correct - thanks for the info!

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

No branches or pull requests

3 participants