Making it easier to work with shaders https://git.yummers.dev/slang.git/atom?h=master 2025-09-19T21:32:28+00:00 2025-09-19T21:32:28+00:00 Gangzheng Tong tonggangzheng@gmail.com 2025-09-19T21:32:28+00:00 urn:sha1:c6d8c6b6890341ac9a849e0f5a96e686731c4e78 This allows doc only changes to PASS the CI by skipping jobs. 2025-09-07T17:07:24+00:00 Gangzheng Tong tonggangzheng@gmail.com 2025-09-07T17:07:24+00:00 urn:sha1:c0d7405d831faa6208b27fe56bf66fb0b138dcc5 1. Adds retry logic - Implements 3-attempt retry mechanism for intermittent test failures 2. Reduces parallelism for Linux - Changes server counts back to 1 for more stable execution, given the test is short in Linux for now 3. Adds detailed error logging - Enhanced diagnostic information for test parsing failures 4. Add Python 3.10 setup for slangpy tests for github runners 5. Removes submodule checkout for slang and slasngpy tests 6. Adds check-ci job - Implements consolidated CI status for simplified branch protection rule --------- Co-authored-by: slangbot <ellieh+slangbot@nvidia.com> Co-authored-by: slangbot <186143334+slangbot@users.noreply.github.com> 2025-09-06T03:31:04+00:00 Gangzheng Tong tonggangzheng@gmail.com 2025-09-06T03:31:04+00:00 urn:sha1:fe87a39e453b64e94446181a9ae5cbfc0f62bf0c - This PR removes "wasm" check from the test job, since there is no test job for "wasm" yet. - Also, move the check for slang-rhi and slangpy test to the job level to skip the setup etc. if no run is needed. - Update the macos compiler version in building.md to match the clang compiler used in CI. - Reduce the server count in Linux and Mac when running slang-test to ease the machine load - Run slangpy test with `-n auto --maxprocesses=4` to speed up --------- Co-authored-by: slangbot <ellieh+slangbot@nvidia.com> Co-authored-by: slangbot <186143334+slangbot@users.noreply.github.com> 2025-09-04T20:08:53+00:00 Gangzheng Tong tonggangzheng@gmail.com 2025-09-04T20:08:53+00:00 urn:sha1:0fb62524bc8beda70694f6c58570ad8096bbe698 The CI has been re-organized with the following: ``` ci.yml (Main Orchestrator) ├── filter job │ ├── Documentation Only? → Yes → Skip CI │ └── Documentation Only? → No → Continue CI │ ├── Build Jobs │ └── ci-slang-build.yml │ ├── common-setup action │ ├── Build & Package │ └── Upload Artifacts (Build package and Tests package) │ └── Test Jobs └── ci-slang-test.yml └── common-test-setup action ├── Download Tests Artifacts ← (from Build) └── Run Tests ``` To achieve fine-grained build->test dependency, instead of using `matrix strategy` in single build (or single test) job, the main ci.yml statically defines the each config of the build and test job. e.g. `build-windows-debug-cl-x86_64-gpu` and `test-windows-debug-cl-x86_64-gpu` are a pair of the build-test job for the windows/debug/ci config. Closes: https://github.com/shader-slang/slang/issues/6728 --------- Co-authored-by: slangbot <ellieh+slangbot@nvidia.com> Co-authored-by: slangbot <186143334+slangbot@users.noreply.github.com> 2025-09-03T00:07:06+00:00 Gangzheng Tong tonggangzheng@gmail.com 2025-09-03T00:07:06+00:00 urn:sha1:7d15d388d7e92547f1cc8ca47748b79589701051 Related to https://github.com/shader-slang/slang/issues/6728 --------- Co-authored-by: slangbot <ellieh+slangbot@nvidia.com> Co-authored-by: slangbot <186143334+slangbot@users.noreply.github.com> 2025-08-29T21:15:52+00:00 Gangzheng Tong tonggangzheng@gmail.com 2025-08-29T21:15:52+00:00 urn:sha1:450ef7934c1adfdf4a3a3c72967de3c5798a020d This PR enables slang-rhi and slangpy tests in the slang CI for MacOS. * exclude the slang-rhi `sampler-array` test for mac; issue tracked in https://github.com/shader-slang/slang/issues/8246 * update slang-rhi for fix of `nested-parameter-block-2` * for slangpy test, install the required python package in the github mac runner only each CI run. Closes: https://github.com/shader-slang/slang/issues/7330 --------- Co-authored-by: slangbot <ellieh+slangbot@nvidia.com> Co-authored-by: slangbot <186143334+slangbot@users.noreply.github.com> 2025-08-26T21:38:57+00:00 Jay Kwak 82421531+jkwak-work@users.noreply.github.com 2025-08-26T21:38:57+00:00 urn:sha1:1681bc67fbae57b54b66c5dcfcbf315d1efa831b fixes https://github.com/shader-slang/slang/issues/8271 This PR does the following, - Fail slang-test when there are VVL error messages. - VVL error for `gfx-unit-test-tool/` were not captured properly by the debug callback. - Set an environment variable, `VK_INSTANCE_LAYERS=VK_LAYER_KHRONOS_validation`, for CI and VisualStudio project setup. - Ignores VVL error about NullHandle is used for the acceleration structure; a fix is at ToT of VVL and not available from release build yet. - Fix VVL error complaining about the varying inputs are not provided for the tests, `gfx-unit-test-tool/linkTimeTypeLayout.internal` and `gfx-unit-test-tool/linkTimeTypeLayoutNested.internal`. --------- Co-authored-by: slangbot <ellieh+slangbot@nvidia.com> Co-authored-by: slangbot <186143334+slangbot@users.noreply.github.com> 2025-08-25T16:08:50+00:00 Jay Kwak 82421531+jkwak-work@users.noreply.github.com 2025-08-25T16:08:50+00:00 urn:sha1:404a8635f4e5a1c7872b8c1cca322904f2ed1579 This commit prints an additional information of GPU version while running CI.yml. This can help us to debug when things don't work as expected. 2025-08-20T16:34:41+00:00 Jay Kwak 82421531+jkwak-work@users.noreply.github.com 2025-08-20T16:34:41+00:00 urn:sha1:a26a11ff5c3f14401bbbbbc4b6e3731056618138 There was a case where a PR passed CI test a long time ago, and when it is merged, it caused a regression. We like to run via-glsl test when it gets merged to prevent it. 2025-08-20T05:39:19+00:00 Harsh Aggarwal (NVIDIA) haaggarwal@nvidia.com 2025-08-20T05:39:19+00:00 urn:sha1:e4a7129b84692ddc3c586f0d0dde95e80e173ed8 Commit bdda8a9 from PR #7862 had to disable some slangy tests in ci.yml