MumbleLink icon indicating copy to clipboard operation
MumbleLink copied to clipboard

Difference in coordinate system conversion between Forge and Fabric mods

Open magneticflux- opened this issue 3 years ago • 2 comments

I recently realized (https://github.com/magneticflux-/fabric-mumblelink-mod/issues/80) that the Forge and Fabric mods have different methods of converting coordinate system handedness. Both methods work, it's just that they result in different left-hand coordinate systems that are incompatible.

Forge: x,z,y https://github.com/zsawyer/MumbleLink/blob/9e2a8db41ed1a49977540f24f343df23cc95d656/mod/src/main/java/zsawyer/mods/mumblelink/mumble/UpdateData.java#L123-L127 Fabric: x,y,-z https://github.com/magneticflux-/fabric-mumblelink-mod/blob/develop/src/main/kotlin/com/skaggsm/mumblelinkmod/client/MathUtils.kt#L12

From Wikipedia:

Interchanging the labels of any two axes reverses the handedness. Reversing the direction of one axis (or of all three axes) also reverses the handedness.


From the documentation:

X increases towards your right, Y increases above your head, and Z increases in front of you

It recommends having "Y" be above the head, which can be preserved by negating either the X or Z axes (The Fabric mod negates the Z axis).

I see two options going forward:

Option 1:

Change the Forge mod to match the Fabric mod.

Pros:

  • Preserves the recommended "Y is up" property

Option 2:

Change the Fabric mod to match the Forge mod.

Pros:

  • Only breaks compatibility with the smaller (I assume) userbase

As it is now, the Fabric and Forge mods are still incompatible due to this issue. It results in confusing behavior because the sounds work in some situations (around 0,0,0), but come from the wrong locations.

magneticflux- avatar Nov 01 '21 21:11 magneticflux-

@zsawyer This should probably be done in the update to 1.18 to avoid breaking compatibility.

Special attention should be paid to https://github.com/zsawyer/MumbleLink/pull/35 to ensure the correct coordinate is offset.

magneticflux- avatar Dec 10 '21 21:12 magneticflux-

Summary: I will change the Forge mod for 1.18

The handedness seems to indeed be the problem. I would like to have the solution be the correct transformation from Minecraft's system to Mumble's system - what ever that may be.

Back in the days when I was developing the mod Mumble and Minecraft would actually use different vectors for "up". Z and Y was simply switched. It seems like that has been changed at some point in time?

From my understanding Y is now up in both MC and Mumble. However Mumble uses a left-handed (as per documentation) and minecraft a right-handed coordinate system (tested using F3 ingame).

I would say that simply transforming for the handedness is the right approach.

While I would have preferred to invert the x-axis - since you already have it done to the z-axis I will adapt the Forge mod accordingly.

zsawyer avatar Dec 12 '21 01:12 zsawyer