{"id":64,"date":"2022-08-01T12:19:49","date_gmt":"2022-08-01T12:19:49","guid":{"rendered":"https:\/\/blogs.scummvm.org\/grisenti\/?p=64"},"modified":"2022-08-01T12:19:49","modified_gmt":"2022-08-01T12:19:49","slug":"debugging-shaders","status":"publish","type":"post","link":"https:\/\/blogs.scummvm.org\/grisenti\/2022\/08\/01\/debugging-shaders\/","title":{"rendered":"Debugging shaders"},"content":{"rendered":"

At the start of this week, graphics with shaders were extremely broken. There were mainly two causes of this. The first was that one of the shader class member functions to set uniform matrices was implemented incorrectly.\u00a0 The other problem was in the shader code. Right now, all of the vertex shaders attributes come from pre-defined attributes. For example, gl_Vertex<\/code> is a 4-component vector containing the vertex position and gl_Color<\/code> contains the vertex color if it has been set. This feature has been deprecated in the more recent version of GLSL but it’s still available in GLSL 1.2, the version being used in the project. The issue was that I thought that the texture coordinates attributes were placed in gl_TexCoord[n]<\/code>, but this variable is used to store the texture coordinates and pass them to the fragment shader, so it doesn’t contain any data. The correct variable was gl_MultiTexCoordn<\/code> (where the last n is the id of the texture coordinate).\u00a0<\/span>\u00a0<\/span><\/p>\n

After solving the previous two, the game was finally playable but with a considerable visual difference from the original.\u00a0<\/span>\u00a0<\/span><\/p>\n

\"picture
original game<\/figcaption><\/figure>\n
\"picture
game in ScummVM<\/figcaption><\/figure>\n

The other problems I found were due to incorrect translation of the original shaders, but these didn\u2019t result in much difference in visual output.\u00a0Unfortunately, I couldn\u2019t find the other problems and for now I moved onto other issues. <\/span>\u00a0<\/span><\/p>\n

On a side note, a helpful tool I used to complete most of the shader debugging has been the Nvidia Nsight graphics debugger. Initially I couldn\u2019t set it up (having sanitizers enabled was one of the problems<\/span><\/span>), and tried to work without it, but I found it almost impossible to guess what the issues where.<\/span>\u00a0<\/span><\/p>\n

\"picture<\/span><\/p>\n

After graphics I moved onto the audio system. The engine used OpenAL with an open-source wrapper also made by Frictional games. ScummVM doesn\u2019t have an equivalent system, but for now I will implement just the simple features, ignoring more complex audio modifiers.\u00a0<\/span>\u00a0<\/span><\/p>\n

Next week I\u2019ll keep working on audio and maybe something else, thought I haven\u2019t made any plans yet.\u00a0<\/span>\u00a0<\/span><\/p>\n","protected":false},"excerpt":{"rendered":"

At the start of this week, graphics with shaders were extremely broken. There were mainly two causes of this. The first was that one of the shader class member functions to set uniform matrices was implemented incorrectly.\u00a0 The other problem was in the shader code. Right now, all of the vertex shaders attributes come from […]<\/p>\n","protected":false},"author":13,"featured_media":0,"comment_status":"open","ping_status":"open","sticky":false,"template":"","format":"standard","meta":{"footnotes":""},"categories":[1],"tags":[],"class_list":["post-64","post","type-post","status-publish","format-standard","hentry","category-uncategorized"],"_links":{"self":[{"href":"https:\/\/blogs.scummvm.org\/grisenti\/wp-json\/wp\/v2\/posts\/64","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/blogs.scummvm.org\/grisenti\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/blogs.scummvm.org\/grisenti\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/blogs.scummvm.org\/grisenti\/wp-json\/wp\/v2\/users\/13"}],"replies":[{"embeddable":true,"href":"https:\/\/blogs.scummvm.org\/grisenti\/wp-json\/wp\/v2\/comments?post=64"}],"version-history":[{"count":1,"href":"https:\/\/blogs.scummvm.org\/grisenti\/wp-json\/wp\/v2\/posts\/64\/revisions"}],"predecessor-version":[{"id":69,"href":"https:\/\/blogs.scummvm.org\/grisenti\/wp-json\/wp\/v2\/posts\/64\/revisions\/69"}],"wp:attachment":[{"href":"https:\/\/blogs.scummvm.org\/grisenti\/wp-json\/wp\/v2\/media?parent=64"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/blogs.scummvm.org\/grisenti\/wp-json\/wp\/v2\/categories?post=64"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/blogs.scummvm.org\/grisenti\/wp-json\/wp\/v2\/tags?post=64"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}