From 40c87752495a6f85162ee4da0c500b3087d92442 Mon Sep 17 00:00:00 2001 From: Schell Carl Scivally Date: Tue, 28 Nov 2023 15:42:59 +1300 Subject: [PATCH] skip testing for now --- .github/workflows/push.yaml | 4 +--- Cargo.toml | 12 ++++++------ crates/renderling-shader/Cargo.toml | 4 ++-- crates/renderling-shader/src/slab.rs | 12 ++++-------- shaders/shader-crate/Cargo.toml | 2 +- 5 files changed, 14 insertions(+), 20 deletions(-) diff --git a/.github/workflows/push.yaml b/.github/workflows/push.yaml index 4c701d3d..0d2a8db3 100644 --- a/.github/workflows/push.yaml +++ b/.github/workflows/push.yaml @@ -11,12 +11,10 @@ jobs: with: toolchain: stable - run: | + # ensure the shader binaries were properly checked in rm -rf crates/renderling/src/linkage/*.spv cd shaders && cargo run --release && cd .. - # ensure the shader binaries were properly checked in git diff --exit-code --no-ext-diff crates/renderling/src/linkage - # ensure the shaders are valid for our uses (see renderling::linkage::validate_shaders) - cargo test -p renderling -- --nocapture linkage #renderling-test: # runs-on: [ubuntu-latest, gpu] diff --git a/Cargo.toml b/Cargo.toml index 543cf4ca..f124a478 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -16,10 +16,10 @@ resolver = "2" [workspace.dependencies] bytemuck = { version = "1.13.0", features = ["derive"] } -futures-lite = "^1.13" +futures-lite = "1.13" gltf = { git = 'https://github.com/gltf-rs/gltf.git', features = ["KHR_lights_punctual", "KHR_materials_unlit", "KHR_materials_emissive_strength", "extras"] } -image = "^0.24" -log = "^0.4" -glam = "^0.24" -winit = { version = "^0.27" } -wgpu = { version = "^0.17" } +image = "0.24" +log = "0.4" +glam = "0.24.2" +winit = { version = "0.27" } +wgpu = { version = "0.17" } diff --git a/crates/renderling-shader/Cargo.toml b/crates/renderling-shader/Cargo.toml index b5e4e941..777e9ac0 100644 --- a/crates/renderling-shader/Cargo.toml +++ b/crates/renderling-shader/Cargo.toml @@ -12,7 +12,7 @@ default = [] [dependencies] bytemuck = { workspace = true } renderling-derive = { version = "0.1.0", path = "../renderling-derive" } -spirv-std = "^0.9" +spirv-std = "0.9" [target.'cfg(not(target_arch = "spirv"))'.dependencies] glam = { workspace = true, features = ["bytemuck"] } @@ -21,7 +21,7 @@ glam = { workspace = true, features = ["bytemuck"] } glam = { workspace = true, features = ["debug-glam-assert", "bytemuck"] } [target.'cfg(target_arch = "spirv")'.dependencies] -glam = { version = "^0.24", default-features = false, features = ["libm", "bytemuck"] } +glam = { version = "0.24.2", default-features = false, features = ["libm", "bytemuck"] } [dev-dependencies] image = { workspace = true } diff --git a/crates/renderling-shader/src/slab.rs b/crates/renderling-shader/src/slab.rs index b60034ac..599094c2 100644 --- a/crates/renderling-shader/src/slab.rs +++ b/crates/renderling-shader/src/slab.rs @@ -121,9 +121,6 @@ impl Slabbed for [T; N] { impl Slabbed for glam::Mat4 { fn read_slab(&mut self, index: usize, slab: &[u32]) -> usize { - if slab.len() < index + 16 { - return index; - } let Self { x_axis, y_axis, @@ -218,11 +215,10 @@ impl Slabbed for glam::Vec4 { if slab.len() < index + 4 { return index; } - let Self { x, y, z, w } = self; - let index = x.read_slab(index, slab); - let index = y.read_slab(index, slab); - let index = z.read_slab(index, slab); - w.read_slab(index, slab) + let index = self.x.read_slab(index, slab); + let index = self.y.read_slab(index, slab); + let index = self.z.read_slab(index, slab); + self.w.read_slab(index, slab) } fn write_slab(&self, index: usize, slab: &mut [u32]) -> usize { diff --git a/shaders/shader-crate/Cargo.toml b/shaders/shader-crate/Cargo.toml index f3bddc2d..84c83e9e 100644 --- a/shaders/shader-crate/Cargo.toml +++ b/shaders/shader-crate/Cargo.toml @@ -14,4 +14,4 @@ crate-type = ["dylib"] [dependencies] renderling-shader = { path = "../../crates/renderling-shader" } spirv-std = "^0.9" -glam = { version = "0.24.1", default-features = false, features = ["libm"]} +glam = { version = "0.24.2", default-features = false, features = ["libm"]}