Compatibility with aframe master threejs r141

[vincentfretin]

When you will try to update your project to use latest aframe master, here is the list of the changes needed in the different libraries so you don't need to search for all the PR I just did :)
THREE.Math was removed in threejs r141, it needs to be replaced with THREE.MathUtils

You need

• networked-aframe 0.10.0 that includes compatibility with threejs r141 networked-aframe/networked-aframe#343
• Use THREE.MathUtils.clamp to be compatible with threejs r141 diarmidmackenzie/christmas-scene#2
• threejs r141 compatibility for movement-controls c-frame/aframe-extras#377

In the meantime, you can also use:

<script src="aframe-master.min.js"></script>
<script>
  THREE.Math = THREE.MathUtils;
</script>

[AdaRoseCannon]

Is there something else in the Master Branch you need? I am happy to update for the sake of updating, though.

• Perhaps we should move the physics engine locally into the project rather than pegging it at a particular version in someone else's project. Just for convenience sake
• We should probably wait for the PR to aframe-extras to be merged then link to the new build

If you want to make a PR I am happy to check it in

[vincentfretin]

No I don't need anything new in aframe master. I was just testing a copy of this project with latest aframe master because I was testing the networked-aframe change for threejs r141 I did.

[AdaRoseCannon]

Cool if there is no hurry we can wait. Thanks for raising PRs in the other repos.

[vincentfretin]

I want to mention here that physx has been extracted to a separate repository https://github.com/c-frame/physx
I merged some aframe-extras PRs, see c-frame/aframe-extras#5
and networked-aframe 0.11.0 was released.

[AdaRoseCannon]

Super!! I don’t really have time to update it myself right now, I’ll get to it when I come back to work although if you have some time to test I’d be happy to take a PR

[vincentfretin]

Before we update the project, we can wait that we make releases for @c-frame/physx, @c-frame/sphere-collider and @c-frame/movement-controls
Also there is a new release for blink controls to remove a warning.
https://cdn.jsdelivr.net/npm/aframe-blink-controls@0.4.1/dist/aframe-blink-controls.min.js

Are you interested in moving your simple-navmesh-constraint component to c-frame organization? We can add you as owner to the github and npm organization. We can create a package @c-frame/simple-navmesh-constraint and publish it to npm. You can do that yourself or I'll probably do it at one point.
If you have some packages you want to move to c-frame organization to allow other maintainers to help you maintaining them, we'll be happy to have you there, it's up to you of course.

[AdaRoseCannon]

Feel free to move it just add an attribution to the file.

[vincentfretin]

FYI in aframe-extras repo, npm run dist generates a sphere-collider.min.js file, so instead of

  <script>window.module = {exports:{}}</script><!-- Dirty hack to get the sphere collider loading :-( -->
  <script src="https://cdn.jsdelivr.net/gh/n5ro/aframe-extras@master/src/misc/sphere-collider.js"></script>

you can use:

  <script src="https://cdn.jsdelivr.net/gh/n5ro/aframe-extras@8c9b13a/dist/components/sphere-collider.min.js"></script>

The current build on master is up to date.

Also aframe 1.4.0 will be released soon.

[vincentfretin]

I see this repository is still using aframe-htmlmesh 1.0.1 and you released 2.0.0, so the html code will need to be updated as well.

I'm just adding notes here while I'm developing my own app where the code diverged a lot from your index.html. I don't have necessary the time right now to update your example and testing it. ;-)

[AdaRoseCannon]

no worries, me either lol

[vincentfretin]

Another fix needed for aframe master:

• Fix RAD2DEG undefined with threejs r144 jure/aframe-blink-controls#22

[vincentfretin]

Also related, this handy-work PR that is already merged but not released:

• frame may be undefined handy-work#15

[vincentfretin]

Fix crash when switching from controllers to hands on Quest (when you put the controllers on the floor):

• gamepad-controls and keyboard-controls fixes c-frame/aframe-extras#385

[vincentfretin]

Fix for aframe-htmlmesh when using dynamically created dom element for the UI on the wrist:

• Proper cleanup when removing html component aframe-htmlmesh#5

[vincentfretin]

Released in aframe-htmlmesh 2.0.1, performance fixes

• Don't create new objects in tick aframe-htmlmesh#7