fu5ha/ultraviolet

Implement `serde::Serialize` and `serde::Deserialize` for bivec and rotor types

thenlevy opened this issue · 7 comments

The serde traits are implemented for the vectors and matrices types. I think it would be usefull to have them also for bivec and rotor types.

I can try to write these implementations, using those of vector and matrices as model.

Yes, I would appreciate this as well.

Actually, on 0.7.1, serde is not used at all (as far as I can see), even for Vec3's.
As de-/serialization is crucial for me, I might opt for another crate, even though I really like this one.

fu5ha commented

Actually, on 0.7.1, serde is not used at all (as far as I can see), even for Vec3's.
As de-/serialization is crucial for me, I might opt for another crate, even though I really like this one.

Support for serialization is gated under the serde feature, as noted in the readme. Currently it supports only scalar vector and matrix types, and the above pr will add support for rotors and bivectors.

Support for serialization is gated under the serde feature, as noted in the readme. Currently it supports only scalar vector and matrix types, and the above pr will add support for rotors and bivectors.

Yes, my bad. I am still new to Rust and don't know how to enable the optional dependencies. I am sorry :-)

fu5ha commented

Support for serialization is gated under the serde feature, as noted in the readme. Currently it supports only scalar vector and matrix types, and the above pr will add support for rotors and bivectors.

Yes, my bad. I am still new to Rust and don't know how to enable the optional dependencies. I am sorry :-)

No problem. See this section of the readme: https://github.com/termhn/ultraviolet#cargo-features

Thank you very much! FIgured it out :-)

fu5ha commented

#75 merged and released as 0.7.2