• Advertisement
Sign in to follow this  

DX11 directx 11 problem switching textures in simple font engine

This topic is 1888 days old which is more than the 365 day threshold we allow for new replies. Please post a new topic.

If you intended to correct an error in the post then please contact us.

Recommended Posts

Hi folks,

I'm trying to display the amount of seconds a program has been running using a simple font engine which samples 2 textures (one for numbers, one for letters). Displaying letters works fine but the numbers come out wrong. By changing the background colors of the textures I've come to the conclusion that I'm not switching shader resources correctly. Can somebody tell me what I'm doing wrong here? Relevant code will most likely be in the DrawString() method.

EDIT: I figure I'm not supposed to switch shader resources before callin Draw(). Comments & suggestion are welcome

FontEngine.cpp

[source lang="cpp"]#include "FontEngine.h"
#include "Graphics.h"
#include "D3DX11.h"
#include "xnamath.h"
struct vertexPos
{
XMFLOAT3 position;
XMFLOAT2 texCoord;
};

FontEngine::FontEngine(Graphics* p_parent)
: m_bInitialized(false),
m_pGraphics(p_parent),
m_pInputLayout(NULL),
m_pVertexShader(NULL),
m_pPixelShader(NULL),
m_pLetterMap(NULL),
m_pNumberMap(NULL),
m_pSampler(NULL),
m_pDynamicVertexBuffer(NULL)
{
}

FontEngine::~FontEngine()
{
cleanup();
}

//remove member objects
void FontEngine::cleanup()
{
m_pGraphics->log(_T("[CLEANING] subsystem FontEngine"));

if(m_pInputLayout)
m_pInputLayout->Release();
if(m_pVertexShader)
m_pVertexShader->Release();
if(m_pPixelShader)
m_pPixelShader->Release();
if(m_pLetterMap)
m_pLetterMap->Release();

if(m_pNumberMap)
m_pNumberMap->Release();
if(m_pSampler)
m_pSampler->Release();
if(m_pDynamicVertexBuffer)
m_pDynamicVertexBuffer->Release();
}

bool FontEngine::initialize()
{
ID3DBlob* vertexShaderBuffer = NULL;
bool compileResult = m_pGraphics->CompileD3DShader(_T("Effects/FontTextureMap.fx"), "VS_Main", "vs_4_0", &vertexShaderBuffer);

if(!compileResult) //texture shader compilation failed
{
MessageBox(NULL, _T("Failed to compile font engine vertex shader"), _T("Fatal Error"), MB_OK);
return false;
}
HRESULT d3dResult;
d3dResult = m_pGraphics->getDevice()->CreateVertexShader(vertexShaderBuffer->GetBufferPointer(),
vertexShaderBuffer->GetBufferSize(),
NULL, &m_pVertexShader);
if(FAILED(d3dResult))
{
MessageBox(NULL, _T("Failed to create font vertex shader"), _T("Fatal Error"), MB_OK);
return false;
}
D3D11_INPUT_ELEMENT_DESC solidColorLayout[] =
{
{ "POSITION", 0, DXGI_FORMAT_R32G32B32_FLOAT, 0, 0, D3D11_INPUT_PER_VERTEX_DATA, 0 },
{ "TEXCOORD", 0, DXGI_FORMAT_R32G32_FLOAT, 0, 12, D3D11_INPUT_PER_VERTEX_DATA, 0}
};
unsigned int totalLayoutElements = ARRAYSIZE( solidColorLayout );

d3dResult = m_pGraphics->getDevice()->CreateInputLayout(solidColorLayout,
totalLayoutElements,
vertexShaderBuffer->GetBufferPointer(),
vertexShaderBuffer->GetBufferSize(),
&m_pInputLayout);
vertexShaderBuffer->Release();

if(FAILED(d3dResult))
{
MessageBox(NULL, _T("Failed to create input layout in fontengine"), _T("Fatal Error"), MB_OK);
return false;
}
//compile and create pixel shader
ID3DBlob* pixelShaderBuffer;
compileResult = m_pGraphics->CompileD3DShader(_T("Effects/FontTextureMap.fx"), "PS_Main", "ps_4_0", &pixelShaderBuffer);

if(!compileResult) //compilation of pixel shader failed
{
MessageBox(NULL, _T("Failed to compile pixel shader in fontengine"), _T("Fatal Error"), MB_OK);
return false;
}

d3dResult = m_pGraphics->getDevice()->CreatePixelShader(pixelShaderBuffer->GetBufferPointer(),
pixelShaderBuffer->GetBufferSize(),
0, &m_pPixelShader);
pixelShaderBuffer->Release();

//warn user if pixel shader could not be created
if(FAILED(d3dResult))
{
MessageBox(NULL, _T("Failed to create pixel shader in fontengine"), _T("Fatal Error"), MB_OK);
return false;
}

//create shader resource view for letters
d3dResult = D3DX11CreateShaderResourceViewFromFile(m_pGraphics->getDevice(), _T("Graphics/Fonts/testFont2.dds"), 0, 0, &m_pLetterMap, 0);
if(FAILED(d3dResult))
{
MessageBox(NULL, _T("Failed to create shader resource view for letters in fontengine"), _T("Fatal Error"), MB_OK);
return false;
}

//create shader resource view for numbers
d3dResult = D3DX11CreateShaderResourceViewFromFile(m_pGraphics->getDevice(), _T("Graphics/Fonts/numbersArial.dds"), 0, 0, &m_pNumberMap, 0);
if(FAILED(d3dResult))
{
MessageBox(NULL, _T("Failed to create shader resource view for numbers in fontengine"), _T("Fatal Error"), MB_OK);
return false;
}


//create sampler state
D3D11_SAMPLER_DESC colorMapDesc;
ZeroMemory(&colorMapDesc, sizeof(colorMapDesc));
colorMapDesc.AddressU = D3D11_TEXTURE_ADDRESS_WRAP;
colorMapDesc.AddressV = D3D11_TEXTURE_ADDRESS_WRAP;
colorMapDesc.AddressW = D3D11_TEXTURE_ADDRESS_WRAP;
colorMapDesc.ComparisonFunc = D3D11_COMPARISON_NEVER;
colorMapDesc.Filter = D3D11_FILTER_MIN_MAG_MIP_LINEAR;
colorMapDesc.MaxLOD = D3D11_FLOAT32_MAX;

d3dResult = m_pGraphics->getDevice()->CreateSamplerState(&colorMapDesc, &m_pSampler);
if(FAILED(d3dResult))
{
MessageBox(NULL, _T("Failed to create sampler state in fontengine"), _T("Fatal Error"), MB_OK);
return false;
}

D3D11_BUFFER_DESC vertexDesc;
ZeroMemory(&vertexDesc, sizeof(vertexDesc));
vertexDesc.Usage = D3D11_USAGE_DYNAMIC;
vertexDesc.CPUAccessFlags = D3D11_CPU_ACCESS_WRITE;
vertexDesc.BindFlags = D3D11_BIND_VERTEX_BUFFER;
const int sizeOfSprite = sizeof(vertexPos) * 6; //six points to a quad
const int maxLetters = 45; //45 quads to a string
vertexDesc.ByteWidth =sizeOfSprite * maxLetters;
//create dynamic buffer
d3dResult = m_pGraphics->getDevice()->CreateBuffer(&vertexDesc, NULL, &m_pDynamicVertexBuffer);
if(FAILED(d3dResult))
{
MessageBox(NULL, _T("Failed to create dynamic buffer in fontengine"), _T("Fatal Error"), MB_OK);
return false;
}

//all checks passed - return true
m_bInitialized = true;
return true;
}

void FontEngine::drawString(tstring p_message, float p_xPosition, float p_yPosition)
{
//TODO datamembers?
//size (in bytes) of a single sprite
const int sizeOfSprite = sizeof(vertexPos) * 6;
const int maxLetters = 45;

int length = p_message.length();

//clamp strings that are too long
if(length > maxLetters)
length = maxLetters;
//per quad two triangles, per triangle three vertices (3*2=6)
const int verticesPerLetter = 6;
D3D11_MAPPED_SUBRESOURCE mapResource;
HRESULT d3dResult = m_pGraphics->getContext()->Map(m_pDynamicVertexBuffer, 0, D3D11_MAP_WRITE_DISCARD, 0, &mapResource);
if(FAILED(d3dResult))
{
MessageBox(NULL, _T("Failed to map dynamic buffer in fontengine"), _T("Fatal Error"), MB_OK);
return;
}

vertexPos* spritePtr = (vertexPos*) mapResource.pData;
//convert to array of characters
const wchar_t* cString = p_message.c_str();
const int indexA = static_cast<char>('A');
const int indexZ = static_cast<char>('Z');
const int index0 = static_cast<char>('0');
const int index9 = static_cast<char>('9');



for(int i = 0; i < length; ++i)
{
//TODO hardcoded!
float charWidth = 32.0f / 800.0f;
// Char's height on screen.
float charHeight = 32.0f / 640.0f;

// Char's texel width.
float texelWidth = 32.0f / 864.0f;

int texLookup = 0; //the "index" of the character in the texture
int letter = static_cast<char>( cString[i] ); //the "index" of the character in the ASCII table
//select letter texture by default because we use space as default when the character isn't found on the fontmap
m_pGraphics->getContext()->PSSetShaderResources( 0, 1, &amp;amp;m_pLetterMap);

if( letter < indexA || letter > indexZ ) //not an uppercase letter?
{
if(letter < index0 || letter > index9) //not a number?
texLookup = ( indexZ - indexA ) + 1; // Grab one index past Z, which is a blank space in the texture.
else //it's a number
{
// Char's texel width.
texelWidth = 32.0f / 333.0f;
texLookup = (letter - index0);
m_pGraphics->getContext()->PSSetShaderResources(1,1, &amp;amp;m_pNumberMap);
}

}
else //uppercase letter
{
//select letter texture by default because we use space as kind of an error character
m_pGraphics->getContext()->PSSetShaderResources( 0, 1, &amp;amp;m_pLetterMap);

// A = 0, B = 1, Z = 25, etc.
texLookup = ( letter - indexA );
}


float thisStartX = p_xPosition + ( charWidth * static_cast<float>( i ) );
float thisEndX = thisStartX + charWidth;
float thisEndY = p_yPosition + charHeight;
spritePtr[0].position = XMFLOAT3( thisEndX, thisEndY, 0.01f );
spritePtr[1].position = XMFLOAT3( thisEndX, p_yPosition, 0.01f );
spritePtr[2].position = XMFLOAT3( thisStartX, p_yPosition, 0.01f );
spritePtr[3].position = XMFLOAT3( thisStartX, p_yPosition, 0.01f );
spritePtr[4].position = XMFLOAT3( thisStartX, thisEndY, 0.01f );
spritePtr[5].position = XMFLOAT3( thisEndX, thisEndY, 0.01f );

float tuStart = 0.0f + ( texelWidth * static_cast<float>( texLookup ) );
float tuEnd = tuStart + texelWidth;
spritePtr[0].texCoord = XMFLOAT2( tuEnd, 0.0f );
spritePtr[1].texCoord = XMFLOAT2( tuEnd, 1.0f );
spritePtr[2].texCoord = XMFLOAT2( tuStart, 1.0f );
spritePtr[3].texCoord = XMFLOAT2( tuStart, 1.0f );
spritePtr[4].texCoord = XMFLOAT2( tuStart, 0.0f );
spritePtr[5].texCoord = XMFLOAT2( tuEnd, 0.0f );
//move forward the size of a single quad (6 vertices per quad)
spritePtr += 6;
}



m_pGraphics->getContext()->Unmap(m_pDynamicVertexBuffer, 0 );
m_pGraphics->getContext()->Draw( 6 * length, 0 );
}
void FontEngine::setupRender()
{
ID3D11DeviceContext* context = m_pGraphics->getContext();
if(context == 0 )
return;

unsigned int stride = sizeof( vertexPos );
unsigned int offset = 0;
context->IASetInputLayout( m_pInputLayout );
context->IASetVertexBuffers( 0, 1, &amp;amp;m_pDynamicVertexBuffer, &amp;amp;stride, &amp;amp;offset );
context->IASetPrimitiveTopology( D3D11_PRIMITIVE_TOPOLOGY_TRIANGLELIST );
context->VSSetShader( m_pVertexShader, 0, 0 );
context->PSSetShader( m_pPixelShader, 0, 0 );
context->PSSetSamplers( 0, 1, &amp;amp;m_pSampler );
}
bool FontEngine::isInitialized()
{
return m_bInitialized;
}[/source] Edited by molehill mountaineer

Share this post


Link to post
Share on other sites
Advertisement
[quote name='MJP' timestamp='1353290990' post='5002202']
Could you post your pixel shader as well?
[/quote]

Sure thing, code is copy-pasted from online source (I wanted to postpone writing pixel shaders for a bit).
In the drawstring() method above I switch shader resources in a for-loop (e.g. 4 quads with letters, 1 with a number, 4 with letters). Only after the loop has finished do I call Draw(). I think this may be what I'm doing wrong - am I supposed to hold two textures in the pixelshader?

[code]
/*
Beginning DirectX 11 Game Programming
By Allen Sherrod and Wendy Jones
Texture Mapping Shader
*/

Texture2D colorMap_ : register( t0 );
SamplerState colorSampler_ : register( s0 );

struct VS_Input
{
float4 pos : POSITION;
float2 tex0 : TEXCOORD0;
};
struct PS_Input
{
float4 pos : SV_POSITION;
float2 tex0 : TEXCOORD0;
};

PS_Input VS_Main( VS_Input vertex )
{
PS_Input vsOut = ( PS_Input )0;
vsOut.pos = vertex.pos;
vsOut.tex0 = vertex.tex0;
return vsOut;
}

float4 PS_Main( PS_Input frag ) : SV_TARGET
{
return colorMap_.Sample( colorSampler_, frag.tex0 );
}
[/code] Edited by molehill mountaineer

Share this post


Link to post
Share on other sites
Seems to me that part of your PSSetShaderResources calls use start slot 0 and part of them use start slot 1 (namely, the number texture is bound to slot 1).

You shader is using only the slot 0 (register t0) , so logically binding resources to slot 1 (register t1) doesn't have any effect.

Cheers!

Share this post


Link to post
Share on other sites
[quote name='kauna' timestamp='1353328982' post='5002325']
Seems to me that part of your PSSetShaderResources calls use start slot 0 and part of them use start slot 1 (namely, the number texture is bound to slot 1).

You shader is using only the slot 0 (register t0) , so logically binding resources to slot 1 (register t1) doesn't have any effect.

Cheers!
[/quote]

Oh okay, so I'd have to add "Texture2D numberMap_ : register( t1 );" in order to actually make this work? Guess I'll have to read up on pixel shaders - thanks!

Share this post


Link to post
Share on other sites
I'm not 100% of the logic of your code, but you could first try to bind the textures to the same slot. Then at least you should be able to see the numbers.

You pixel shader calls Sample with colorMap_ map which is using register t0. Adding "Texture2D numberMap_ : register( t1 );" Doesn't do anything alone, you'll need to call Sample with that texture also, but that doesn't really lead to anywhere in this case.

Cheers!

Share this post


Link to post
Share on other sites
Sign in to follow this  

  • Advertisement
  • Advertisement
  • Popular Now

  • Advertisement
  • Similar Content

    • By AxeGuywithanAxe
      I wanted to see how others are currently handling descriptor heap updates and management.
      I've read a few articles and there tends to be three major strategies :
      1 ) You split up descriptor heaps per shader stage ( i.e one for vertex shader , pixel , hull, etc)
      2) You have one descriptor heap for an entire pipeline
      3) You split up descriptor heaps for update each update frequency (i.e EResourceSet_PerInstance , EResourceSet_PerPass , EResourceSet_PerMaterial, etc)
      The benefits of the first two approaches is that it makes it easier to port current code, and descriptor / resource descriptor management and updating tends to be easier to manage, but it seems to be not as efficient.
      The benefits of the third approach seems to be that it's the most efficient because you only manage and update objects when they change.
    • By evelyn4you
      hi,
      until now i use typical vertexshader approach for skinning with a Constantbuffer containing the transform matrix for the bones and an the vertexbuffer containing bone index and bone weight.
      Now i have implemented realtime environment  probe cubemaping so i have to render my scene from many point of views and the time for skinning takes too long because it is recalculated for every side of the cubemap.
      For Info i am working on Win7 an therefore use one Shadermodel 5.0 not 5.x that have more options, or is there a way to use 5.x in Win 7
      My Graphic Card is Directx 12 compatible NVidia GTX 960
      the member turanszkij has posted a good for me understandable compute shader. ( for Info: in his engine he uses an optimized version of it )
      https://turanszkij.wordpress.com/2017/09/09/skinning-in-compute-shader/
      Now my questions
       is it possible to feed the compute shader with my orignial vertexbuffer or do i have to copy it in several ByteAdressBuffers as implemented in the following code ?
        the same question is about the constant buffer of the matrixes
       my more urgent question is how do i feed my normal pipeline with the result of the compute Shader which are 2 RWByteAddressBuffers that contain position an normal
      for example i could use 2 vertexbuffer bindings
      1 containing only the uv coordinates
      2.containing position and normal
      How do i copy from the RWByteAddressBuffers to the vertexbuffer ?
       
      (Code from turanszkij )
      Here is my shader implementation for skinning a mesh in a compute shader:
      1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 struct Bone { float4x4 pose; }; StructuredBuffer<Bone> boneBuffer;   ByteAddressBuffer vertexBuffer_POS; // T-Pose pos ByteAddressBuffer vertexBuffer_NOR; // T-Pose normal ByteAddressBuffer vertexBuffer_WEI; // bone weights ByteAddressBuffer vertexBuffer_BON; // bone indices   RWByteAddressBuffer streamoutBuffer_POS; // skinned pos RWByteAddressBuffer streamoutBuffer_NOR; // skinned normal RWByteAddressBuffer streamoutBuffer_PRE; // previous frame skinned pos   inline void Skinning(inout float4 pos, inout float4 nor, in float4 inBon, in float4 inWei) {  float4 p = 0, pp = 0;  float3 n = 0;  float4x4 m;  float3x3 m3;  float weisum = 0;   // force loop to reduce register pressure  // though this way we can not interleave TEX - ALU operations  [loop]  for (uint i = 0; ((i &lt; 4) &amp;&amp; (weisum&lt;1.0f)); ++i)  {  m = boneBuffer[(uint)inBon].pose;  m3 = (float3x3)m;   p += mul(float4(pos.xyz, 1), m)*inWei;  n += mul(nor.xyz, m3)*inWei;   weisum += inWei;  }   bool w = any(inWei);  pos.xyz = w ? p.xyz : pos.xyz;  nor.xyz = w ? n : nor.xyz; }   [numthreads(1024, 1, 1)] void main( uint3 DTid : SV_DispatchThreadID ) {  const uint fetchAddress = DTid.x * 16; // stride is 16 bytes for each vertex buffer now...   uint4 pos_u = vertexBuffer_POS.Load4(fetchAddress);  uint4 nor_u = vertexBuffer_NOR.Load4(fetchAddress);  uint4 wei_u = vertexBuffer_WEI.Load4(fetchAddress);  uint4 bon_u = vertexBuffer_BON.Load4(fetchAddress);   float4 pos = asfloat(pos_u);  float4 nor = asfloat(nor_u);  float4 wei = asfloat(wei_u);  float4 bon = asfloat(bon_u);   Skinning(pos, nor, bon, wei);   pos_u = asuint(pos);  nor_u = asuint(nor);   // copy prev frame current pos to current frame prev pos streamoutBuffer_PRE.Store4(fetchAddress, streamoutBuffer_POS.Load4(fetchAddress)); // write out skinned props:  streamoutBuffer_POS.Store4(fetchAddress, pos_u);  streamoutBuffer_NOR.Store4(fetchAddress, nor_u); }  
    • By mister345
      Hi, can someone please explain why this is giving an assertion EyePosition!=0 exception?
       
      _lightBufferVS->viewMatrix = DirectX::XMMatrixLookAtLH(XMLoadFloat3(&_lightBufferVS->position), XMLoadFloat3(&_lookAt), XMLoadFloat3(&up));
      It looks like DirectX doesnt want the 2nd parameter to be a zero vector in the assertion, but I passed in a zero vector with this exact same code in another program and it ran just fine. (Here is the version of the code that worked - note XMLoadFloat3(&m_lookAt) parameter value is (0,0,0) at runtime - I debugged it - but it throws no exceptions.
          m_viewMatrix = DirectX::XMMatrixLookAtLH(XMLoadFloat3(&m_position), XMLoadFloat3(&m_lookAt), XMLoadFloat3(&up)); Here is the repo for the broken code (See LightClass) https://github.com/mister51213/DirectX11Engine/blob/master/DirectX11Engine/LightClass.cpp
      and here is the repo with the alternative version of the code that is working with a value of (0,0,0) for the second parameter.
      https://github.com/mister51213/DX11Port_SoftShadows/blob/master/Engine/lightclass.cpp
    • By mister345
      Hi, can somebody please tell me in clear simple steps how to debug and step through an hlsl shader file?
      I already did Debug > Start Graphics Debugging > then captured some frames from Visual Studio and
      double clicked on the frame to open it, but no idea where to go from there.
       
      I've been searching for hours and there's no information on this, not even on the Microsoft Website!
      They say "open the  Graphics Pixel History window" but there is no such window!
      Then they say, in the "Pipeline Stages choose Start Debugging"  but the Start Debugging option is nowhere to be found in the whole interface.
      Also, how do I even open the hlsl file that I want to set a break point in from inside the Graphics Debugger?
       
      All I want to do is set a break point in a specific hlsl file, step thru it, and see the data, but this is so unbelievably complicated
      and Microsoft's instructions are horrible! Somebody please, please help.
       
       
       

    • By mister345
      I finally ported Rastertek's tutorial # 42 on soft shadows and blur shading. This tutorial has a ton of really useful effects and there's no working version anywhere online.
      Unfortunately it just draws a black screen. Not sure what's causing it. I'm guessing the camera or ortho matrix transforms are wrong, light directions, or maybe texture resources not being properly initialized.  I didnt change any of the variables though, only upgraded all types and functions DirectX3DVector3 to XMFLOAT3, and used DirectXTK for texture loading. If anyone is willing to take a look at what might be causing the black screen, maybe something pops out to you, let me know, thanks.
      https://github.com/mister51213/DX11Port_SoftShadows
       
      Also, for reference, here's tutorial #40 which has normal shadows but no blur, which I also ported, and it works perfectly.
      https://github.com/mister51213/DX11Port_ShadowMapping
       
  • Advertisement