nalgebra support: ShaderSize is not implemented for Matrix<f32, Const<3>... about encase HOT 10 CLOSED

Do you need it released or could you use a git dependency for now?

@teoxoy nope. I added the following to Cargo.toml and the changes in #23 do resolve the problem:

[patch.crates-io]
encase = { git = "https://github.com/teoxoy/encase.git", rev = "e35c7793a6126f5677b31bcfeec4ea3701cc2348" }

from encase.

I think this should work. Do you have the nalgebra feature flag of encase switched on?

from encase.

@teoxoy yes:

encase = { version = "0.4.1", features = ["nalgebra"] }

from encase.

I'm seeing code for Vector4<T> and vector slices in the nalgebra implementations but not SVector. Is this just a missing implementation?
https://github.com/teoxoy/encase/blob/main/src/impls/nalgebra.rs

from encase.

I'm seeing code for Vector4 and vector slices in the nalgebra implementations but not SVector. Is this just a missing implementation?

@ScanMountGoat AFAIK Vector is an alias for SVector:

https://nalgebra.org/docs/user_guide/vectors_and_matrices

• SVector<T, D> is a statically-sized column vector with D rows, e.g., SVector<f32, 10>.

[...]

Aliases exist for statically-sized vectors and matrices with small dimensions:

• Vector1 .. Vector6: are column vectors of dimension 1 to 6.

That's why I would expect it to work.

from encase.

I'm seeing code for Vector4 and vector slices in the nalgebra implementations but not SVector. Is this just a missing implementation?

@ScanMountGoat AFAIK Vector is an alias for SVector:

Looking at nalgebra's source:

pub type SVector<T, const D: usize> = Matrix<T, Const<D>, U1, ArrayStorage<T, D, 1>>;
pub type Vector4<T> = Matrix<T, U4, U1, ArrayStorage<T, 4, 1>>;

You are using SVector<f32, 4> and encase uses Vector4<T> - they should point to the same definition.

Do you have a minimum reproducible example?

from encase.

Do you have a minimum reproducible example?

@teoxoy here you go:

https://github.com/JMLX42/wgsl_to_wgpu/tree/bug/encase-22/example

from encase.

Ah, I think I figured it out. I haven't updated encase to support nalgebra v0.32, but you request it in Cargo.toml.
You essentially have both versions in your project now.

from encase.

Ah, I think I figured it out. I haven't updated encase to support nalgebra v0.32, but you request it in Cargo.toml.

@teoxoy here you go: #23

Not sure if fixes the problem though...

from encase.

The next major release (0.5) will contain the PR. Do you need it released or could you use a git dependency for now?

from encase.