webxr-input-profiles icon indicating copy to clipboard operation
webxr-input-profiles copied to clipboard

Latest version of hands doesn't have skinning anymore

Open hybridherbst opened this issue 3 years ago • 13 comments

Profile id generic-hand

Describe the bug Looks like while in https://cdn.jsdelivr.net/npm/@webxr-input-profiles/[email protected]/dist/profiles/generic-hand/, left.glb and right.glb both are ~90kB and contain skinning information, in the latest version in this repo they don't have that anymore (and are ~60kB). I also ran npm run build but I think that just copies them over and does not modify them (size stays ~60kB and they don't have skinning info).

To Reproduce Steps to reproduce the behavior:

  1. Go to https://cdn.jsdelivr.net/npm/@webxr-input-profiles/[email protected]/dist/profiles/generic-hand/ (the one referenced by three.js when fetching hand models)
  2. download left.glb
  3. Check with any glTF viewer for the existance of skinning information
  4. Note that it has skinning information
  5. Check with latest left.glb and right.glb
  6. They have a rotation change (https://github.com/immersive-web/webxr-input-profiles/pull/226) but also don't have skinning information anymore.

Expected behavior Skinning :)

Screenshots 1.0.12 (with skinning): image

Now (no skinning): image

hybridherbst avatar Oct 24 '22 12:10 hybridherbst

Looks like this information was stripped out during an optimization step. Want to look at my pull request and see if it has the information you expect? https://github.com/immersive-web/webxr-input-profiles/pull/235

intrepidOlivia avatar Oct 25 '22 16:10 intrepidOlivia

@hybridherbst can you verify that this is working for you now?

cabanier avatar Oct 31 '22 18:10 cabanier

Skinning data is there, but bones are now all rotated @cabanier

com oculus shellenv-20221031-210525

I believe you ran into the same Blender trap as I did a while ago for the "Monster Hands": https://twitter.com/hybridherbst/status/1584524129054904321

Is there a good place to add this kind of info right in this repository? E.g. https://github.com/immersive-web/webxr-input-profiles/blob/main/packages/assets/tutorial/README.md?

hybridherbst avatar Oct 31 '22 20:10 hybridherbst

If you want, you can quickly test by

  • going to https://glitch.com/edit/#!/three-xr?path=index.html%3A220%3A100
  • remixing it
  • dropping your left.glb and right.glb into "assets"
  • copying the path into index.html:220 (that's how I just tested your new models)

hybridherbst avatar Oct 31 '22 20:10 hybridherbst

Is there a good place to add this kind of info right in this repository? E.g. https://github.com/immersive-web/webxr-input-profiles/blob/main/packages/assets/tutorial/README.md?

I'm not sure how frequently that tutorial is referenced, but this kind of exporting tip definitely fits well with the rest of the content in that document. Feel free to put up a pull request to add it, because I feel that's a valuable bit of info!

toji avatar Oct 31 '22 23:10 toji

Sorry for the delay on this, I've updated pull request https://github.com/immersive-web/webxr-input-profiles/pull/235 to include skinning while maintaining proper rotation and bone placement.

intrepidOlivia avatar Dec 02 '22 18:12 intrepidOlivia

Sorry for the delay on this, I've updated pull request #235 to include skinning while maintaining proper rotation and bone placement.

Have you verified that the hand still work in xr?

cabanier avatar Dec 02 '22 18:12 cabanier

Have you verified that the hand still work in xr?

I personally did, a second source of verification would be welcome.

intrepidOlivia avatar Dec 02 '22 20:12 intrepidOlivia

Have you verified that the hand still work in xr?

I personally did, a second source of verification would be welcome.

@hybridherbst can you try? You found the original problem

cabanier avatar Dec 03 '22 08:12 cabanier

Can confirm it works as hands again! But I'm not sure about the orientation when using it as default controller now (which I think was the original intent behind changing them in the first place)

hybridherbst avatar Dec 03 '22 17:12 hybridherbst

Can confirm it works as hands again! But I'm not sure about the orientation when using it as default controller now (which I think was the original intent behind changing them in the first place)

Are they oriented the wrong way again?

cabanier avatar Dec 03 '22 18:12 cabanier

It has been some time since @hybridherbst identified an issue with the mesh. I can verify that these GLB files function correctly. However, when attempting to create custom hand designs by importing these models into Blender, I encounter a challenge. While I can modify the material without issues, upon re-exporting them as GLB, there seems to be a misalignment in their orientation. Could anyone provide guidance on how to import/export from Blender, please? before after https://youtube.com/shorts/uKJGmGdi7mk?si=2BxVdYHv3UVnJ3sp

sylwesterdigital avatar Aug 26 '23 14:08 sylwesterdigital

@sylwesterdigital you may want to check https://twitter.com/hybridherbst/status/1584524129054904321 - the defaults for Blender glTF import are wrong, you need to change them if you plan to export as glTF again.

EDIT: Ah, I see you found it already :)

hybridherbst avatar Aug 26 '23 15:08 hybridherbst

I believe this issue can be closed as the problem has been resolved for a while. Thanks all!

hybridherbst avatar Mar 20 '24 10:03 hybridherbst