jrbudda / Vivecraft_117

VR mod for Minecraft. For Version 1.17.X http://www.vivecraft.org
Other
57 stars 7 forks source link

"Reverse Hands" option needs work #27

Open cmbruns opened 2 years ago

cmbruns commented 2 years ago

This is a follow on to an issue in one of the older vivecraft repos https://github.com/jrbudda/Vivecraft_110/issues/105

Expected Behavior:

  1. The "main hand" tool appears on the same side in both VR and non-VR play
  2. A third-person observer sees the same main-hand side and orientation as the VR player does.
  3. There should be just one setting for the "main hand" leftiness.

Actual Behavior:

  1. In some cases, the "main hand" tool appears on different sides of the body in VR vs. non-VR play. The two play modes should be consistent.
  2. In some cases, the "main hand" tool or pose appearance differs between first-person and third-person views. The hands should appear consistently when viewed by one's self, and by other players.
  3. There are two Options: Skin Customization->Main Hand: [Right, Left] and VR Settings->Controller Settings->Reverse Hands: [OFF, ON]. The main tool hand should be set in just one place.

Test Conditions:

Test Results:

Skin Customization VR Settings 1st person VR 1st person non-VR 3rd person
Main Hand: Right Reverse Hands: OFF vr_first_right :heavy_check_mark: nonvr_first_right :heavy_check_mark: third_right_off :heavy_check_mark:
Main Hand: Right Reverse Hands: ON vr_first_right_on :x: nonvr_first_right :heavy_check_mark: third_right_on
Main Hand: Left Reverse Hands: OFF vr_first_left :x: nonvr_first_left :heavy_check_mark: third_left_off
Main Hand: Left Reverse Hands: ON vr_first_left_on :x: nonvr_first_left :heavy_check_mark: third_left_on :x:

What is working correctly:

What is not working correctly:

There are interesting changes in the arm textures in the various cases, but I have not yet studied carefully enough to decide if any are incorrect.

Techjar commented 2 years ago

Yep.

Techjar commented 2 years ago

This is all going to get restructured at some point, and we'll either completely ignore the vanilla hand setting, or merge the reverse hands setting into it. Having two separate settings is dumb.

jrbudda commented 2 years ago

thanks for the very thorough analysis, this will serve as a useful test matrix for when we tackle this problem