Modifying GeoVector to support GLM. about novelrt HOT 5 CLOSED

It might be beneficial to have GeoVector wrap it for certain functionality. Then it is easily swappable without taking a direct dependency (in the public surface area).

from novelrt.

I think Tanner's idea would be good here. That, and GLM wouldn't hinder a Vulkan implementation as Vulkan has no linear algebra operations (glm is actually recommended in this case), but it'd be useless for a DX implementation if i recall correctly.

from novelrt.

Ok, so then the question is, how much of a priority is it? Things are working as of PR #23 so I'd argue this isn't an immediate essential as we have other concerns such as text to implement.

from novelrt.

This should be lower priority IMO. If we can get the basic implementation concerns done without needing to wrap GeoVector or modify it much, then we should be okay to push it back.

However, if there comes time where you will need to start adding a lot more to GeoVector and glm would assist with this, then we'd revisit. (This part I admit I'm unsure of since I'm not as familiar with gfx programming)

from novelrt.

Due to this being now required it has already been added to the codebase in #26 . Closing.

from novelrt.