The source code for The Vulkan-LoaderAndValidationLayer components is sponsored by Khronos and LunarG.
The Vulkan validation layers are one of the larger and more important components in this repository. While there are often active and organized development efforts underway to improve their coverage, there are always opportunities for anyone to help by contributing additional validation layer checks and tests for these validation checks.
There are a couple of methods to identify areas of need:
Of course, if you have your own work in mind, please open an issue to describe it and assign it to yourself. Finally, please feel free to contact any of the developers that are actively contributing should you wish to coordinate further. Please see the section about Validation Layers later on this page.
Repository Issue labels:
It is the maintainers goal for all issues to be assigned within one business day of their submission. If you choose to work on an issue that is assigned, simply coordinate with the current assignee.
Use the Google style guide for source code with the following exceptions:
Run clang-format on your changes to maintain consistent formatting
.clang-format files
present in the repository to define clang-format settings which are found and used automatically by clang-format.# Make changes to the source. $ git add -u . $ git clang-format --style=file # Check to see if clang-format made any changes and if they are OK. $ git add -u . $ git commit
Strive for commits that implement a single or related set of functionality, using as many commits as is necessary (more is better). That said, please ensure that the repository compiles and passes tests without error for each commit in your pull request. Note that to be accepted into the repository, the pull request must [pass all tests](#testing your changes) on all supported platforms -- the automatic Github Travis and AppVeyor continuous integration features will assist in enforcing this requirement.
Run the existing tests in the repository before and after each if your commits to check for any regressions. There are some tests that appear in all repositories. These tests can be found in the following folders inside of your target build directory:
(These instructions are for Linux)
In the demos
directory, run:
cube cube --validate smoke smoke --validate vulkaninfo
tests
directory, run:run_all_tests.sh
On Windows, a quick sanity check can be run from inside Visual Studio -- just run the vk_layer_validation_tests
project, or you can run run_all_tests.ps1
from a PowerShell window
Note that some tests may fail with known issues or driver-specific problems. The idea here is that your changes should not change the test results, unless that was the intent of your changes.
Run tests that explicitly exercise your changes.
Feel free to subject your code changes to other tests as well!
tests
directory and contributed at the same time as the new validation check itself, along with appropriate updates to layers\vk_validation_error_database.txt
. There are many existing validation tests in this directory that can be used as a starting point.You will be prompted with a one-time "click-through" CLA dialog as part of submitting your pull request or other contribution to GitHub.
All contributions made to the Vulkan-LoaderAndValidationLayers repository are Khronos branded and as such, any new files need to have the Khronos license (Apache 2.0 style) and copyright included. Please see an existing file in this repository for an example.
All contributions made to the LunarG repositories are to be made under the Apache 2.0 license and any new files need to include this license and any applicable copyrights.
You can include your individual copyright after any existing copyrights.