{"action":"created","issue":{"url":"https://api.github.com/repos/KhronosGroup/Vulkan-ValidationLayers/issues/52","repository_url":"https://api.github.com/repos/KhronosGroup/Vulkan-ValidationLayers","labels_url":"https://api.github.com/repos/KhronosGroup/Vulkan-ValidationLayers/issues/52/labels{/name}","comments_url":"https://api.github.com/repos/KhronosGroup/Vulkan-ValidationLayers/issues/52/comments","events_url":"https://api.github.com/repos/KhronosGroup/Vulkan-ValidationLayers/issues/52/events","html_url":"https://github.com/KhronosGroup/Vulkan-ValidationLayers/issues/52","id":322648090,"number":52,"title":"Core validation: Misleading error reported if no pipeline is bound and app tries to change a dynamic state","user":{"login":"karl-lunarg","id":16430219,"avatar_url":"https://avatars2.githubusercontent.com/u/16430219?v=4","gravatar_id":"","url":"https://api.github.com/users/karl-lunarg","html_url":"https://github.com/karl-lunarg","followers_url":"https://api.github.com/users/karl-lunarg/followers","following_url":"https://api.github.com/users/karl-lunarg/following{/other_user}","gists_url":"https://api.github.com/users/karl-lunarg/gists{/gist_id}","starred_url":"https://api.github.com/users/karl-lunarg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/karl-lunarg/subscriptions","organizations_url":"https://api.github.com/users/karl-lunarg/orgs","repos_url":"https://api.github.com/users/karl-lunarg/repos","events_url":"https://api.github.com/users/karl-lunarg/events{/privacy}","received_events_url":"https://api.github.com/users/karl-lunarg/received_events","type":"User","site_admin":false},"labels":[{"id":921967308,"url":"https://api.github.com/repos/KhronosGroup/Vulkan-ValidationLayers/labels/enhancement","name":"enhancement","color":"a2eeef","default":true}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/KhronosGroup/Vulkan-ValidationLayers/milestones/3","html_url":"https://github.com/KhronosGroup/Vulkan-ValidationLayers/milestone/3","labels_url":"https://api.github.com/repos/KhronosGroup/Vulkan-ValidationLayers/milestones/3/labels","id":3342530,"number":3,"title":"P1","description":"","creator":{"login":"karl-lunarg","id":16430219,"avatar_url":"https://avatars2.githubusercontent.com/u/16430219?v=4","gravatar_id":"","url":"https://api.github.com/users/karl-lunarg","html_url":"https://github.com/karl-lunarg","followers_url":"https://api.github.com/users/karl-lunarg/followers","following_url":"https://api.github.com/users/karl-lunarg/following{/other_user}","gists_url":"https://api.github.com/users/karl-lunarg/gists{/gist_id}","starred_url":"https://api.github.com/users/karl-lunarg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/karl-lunarg/subscriptions","organizations_url":"https://api.github.com/users/karl-lunarg/orgs","repos_url":"https://api.github.com/users/karl-lunarg/repos","events_url":"https://api.github.com/users/karl-lunarg/events{/privacy}","received_events_url":"https://api.github.com/users/karl-lunarg/received_events","type":"User","site_admin":false},"open_issues":17,"closed_issues":0,"state":"open","created_at":"2018-05-14T01:35:37Z","updated_at":"2018-05-14T01:44:21Z","due_on":null,"closed_at":null},"comments":3,"created_at":"2018-05-14T01:44:20Z","updated_at":"2018-05-14T01:44:24Z","closed_at":null,"author_association":"CONTRIBUTOR","body":"**Issue by [DominikWitczakAMD](https://github.com/DominikWitczakAMD) (MIGRATED)**\n_Monday Oct 16, 2017 at 09:27 GMT_\n_Originally opened as https://github.com/KhronosGroup/Vulkan-LoaderAndValidationLayers/issues/2135_\n\n----\n\nConsider the use case below:\r\n\r\n1. Start command buffer recording.\r\n2. Set dynamic scissor data.\r\n\r\nOf course, there's a missing 'bind pipeline' call above which is what should be raised by the layer. However, what I'm seeing with a ToT version of core validation layer is:\r\n\r\n``\r\nvkCmdSetScissor(): pipeline was created without VK_DYNAMIC_STATE_SCISSOR flag. The spec valid usage text states 'The currently bound graphics pipeline must have been created with the VK_DYNAMIC_STATE_SCISSOR dynamic state enabled' (https://www.khronos.org/registry/vulkan/specs/1.0/html/vkspec.html#VUID-vkCmdSetScissor-None-00590). \r\n``\r\n\r\nWhat I believe it should say should be along the lines of:\r\n\r\n``\r\nYou've got no GFX pipeline bound, so that vkCmdSetScissor() call of yours makes no sense. Go away.\r\n``\n"},"comment":{"url":"https://api.github.com/repos/KhronosGroup/Vulkan-ValidationLayers/issues/comments/388674334","html_url":"https://github.com/KhronosGroup/Vulkan-ValidationLayers/issues/52#issuecomment-388674334","issue_url":"https://api.github.com/repos/KhronosGroup/Vulkan-ValidationLayers/issues/52","id":388674334,"user":{"login":"karl-lunarg","id":16430219,"avatar_url":"https://avatars2.githubusercontent.com/u/16430219?v=4","gravatar_id":"","url":"https://api.github.com/users/karl-lunarg","html_url":"https://github.com/karl-lunarg","followers_url":"https://api.github.com/users/karl-lunarg/followers","following_url":"https://api.github.com/users/karl-lunarg/following{/other_user}","gists_url":"https://api.github.com/users/karl-lunarg/gists{/gist_id}","starred_url":"https://api.github.com/users/karl-lunarg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/karl-lunarg/subscriptions","organizations_url":"https://api.github.com/users/karl-lunarg/orgs","repos_url":"https://api.github.com/users/karl-lunarg/repos","events_url":"https://api.github.com/users/karl-lunarg/events{/privacy}","received_events_url":"https://api.github.com/users/karl-lunarg/received_events","type":"User","site_admin":false},"created_at":"2018-05-14T01:44:24Z","updated_at":"2018-05-14T01:44:24Z","author_association":"CONTRIBUTOR","body":"<a href=\"https://github.com/krOoze\"><img src=\"https://avatars1.githubusercontent.com/u/3791331?v=4\" align=\"left\" width=\"48\" height=\"48\" hspace=\"10\"></img></a> **Comment by [krOoze](https://github.com/krOoze) (MIGRATED)**\n_Monday Nov 27, 2017 at 12:41 GMT_\n\n----\n\n@DominikWitczakAMD That statement was added later to the spec, potentially introducing spec bug (and at the minimum incompatible change).\r\nAlso consider this:\r\n\r\n> Dynamic state not set by binding the pipeline object **can** be modified **at any time** and **persists for the lifetime** of the **command buffer**, or until modified by another dynamic state command or another pipeline bind.\r\n\r\nNotice, that the dynamic state is a state of the command buffer, and can be changed at any time during its lifetime.\r\n\r\nOr:\r\n\r\n> Modifying dynamic state that is not set as dynamic by the pipeline state object will lead to undefined results.\r\n\r\nNotice it assumes the state can be modified if non-dynamic pipeline is bound. Notice \"undefined results\", not \"undefined behavior\".\r\n\r\nI simply do not have the confidence the VU is correct if the spec is not self-consistent on this point...\n"}}
{"actor":{"display_login":"karl-lunarg"}}