require("THREE") should be require("three")

[jedleslie]

Hi - Thanks for the great library! I'm using three-geo along side three.js in my application. dist/three-geo.min.js has require('THREE') which webkit will not compile as it is not the right case. Changing the min.js to use require("three") works just fine.

If this is just a setup issue on my side I apologize - but typically npm packages are all lower case. Might make sense to update this.

Thanks!

[j-devel]

Thanks for reporting! When using 'dist/three-geo.min.js', we need to load it as tag:<script src="dist/three-geo.min.js"></script> (c.f. Setup section). Also, it assumes the external symbol 'THREE' is already set up by three.js. That's why 'THREE'.

Changing the min.js to use require("three") works just fine.

I'm not sure about your setup, but if you are using some sort of bundler, you might want to try the ES module version: 'dist/three-geo.esm.js'.