8thwall / web

8th Wall Web projects and resources.
515 stars 326 forks source link

Update matrix inversion method due to deprecation #165

Closed positlabs closed 3 years ago

positlabs commented 3 years ago

getInverse() is deprecated and it junks up the console with a new warning on every frame update. This PR uses the new recommended method for getting the inverse matrix

atomarch commented 3 years ago

Thanks for the contribution @positlabs! Looks like this file is slightly out of date, and the XR8.Threejs.pipelineModule() that is built into 8th Wall's AR engine already has this fix (plus some extra logic for backwards compatibility with older versions of three.js)

I'll merge this but then follow it up with another update so it more closely follows the built-in module.