nalgebra upgrade

[hdhoang]

due to rust-lang/rust#49799.

I hit a wierd failure with this example:

error[E0308]: mismatched types
  --> examples/show_in_emerald.rs:81:9
   |
81 |         indices,
   |         ^^^^^^^ expected u16, found u32
   |
   = note: expected type `std::vec::Vec<nalgebra::Point<u16, _>>`
              found type `std::vec::Vec<nalgebra::Point<u32, _>>`

I think indices are GLuint, which should still be c_uint/u32. Could you take a closer look?

[Libbum]

Thanks for this. I'm travelling at the moment so won't get to it until tomorrow, but looks good on first glance.

I really should get back to finishing this package properly. Have you found a use for it? Any features you would like added?

[hdhoang]

Hey, no worry. I'm coming at this from the rust-lang PR, I'm not a user though.

Travis also fails on u16/u32 like above: https://travis-ci.com/hdhoang/spherical-cow/builds/89428375#L1033. I have no idea how that comes up.

[Libbum]

Sure thing. Thanks for the tip-off! Will take a deeper look tomorrow.

[Libbum]

The reason for the failure in emerald was from sebcrozet/kiss3d#130 - WebGL 1.0 compatibility has dropped the size of the faces uint.

[codecov]

Codecov Report

Merging #17 into master will not change coverage.
The diff coverage is 100%.

@@         Coverage Diff          @@
##           master   #17   +/-   ##
====================================
  Coverage      80%   80%           
====================================
  Files           6     6           
  Lines         385   385           
====================================
  Hits          308   308           
  Misses         77    77

Impacted Files	Coverage Δ
src/lib.rs	98.78% <100%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6828192...b6d67ba. Read the comment docs.

[hdhoang]

great to know! thanks for accepting this