Vivecraft / VivecraftMod

An experimental port of Vivecraft to Forge/Fabric
Other
162 stars 33 forks source link

Controlling a horse via the reins lack visual feedback. #260

Open kevina opened 4 months ago

kevina commented 4 months ago

In 1.12 it is obvious how your hands control the reins after you mount a house thanks to the "much better cosmetic display" introduced in 1.1.12 v5. Basically the rein where attached to your hands and not just the horse. Newer versions lack this visual feedback making controlling a horse confusing.

Originally reported against 1.16 and still and issue with 1.20: https://github.com/jrbudda/Vivecraft_116/issues/13

fayer3 commented 4 months ago

what is a rein?

fayer3 commented 4 months ago

do you have a screenshot of what you mean?

kevina commented 4 months ago

what is a rein?

what is used to control the horse in real life, see https://www.merriam-webster.com/dictionary/rein for an illustration

do you have a screenshot of what you mean?

not at the moment, but if you try and ride a horse in 1.12 it will become obvious what I mean

fayer3 commented 4 months ago

screenshot for reference grafik and code links https://github.com/jrbudda/Vivecraft_112/blob/master/patches/net/minecraft/client/model/ModelHorse.java.patch

you only want the visual rein, or also the control stuff it comes with?

fayer3 commented 4 months ago

was removed in 1.15 btw, probably because of the rendering changes, and then forgotten

kevina commented 4 months ago

The controller stuff is still there, but it lacks the visual feedback. I could not figure out how to ride a horse until I played 1.12.

fayer3 commented 4 months ago

no it's not. the horse tracker is non functional atm, it's just the same as regular movement, point an walk https://github.com/Vivecraft/VivecraftMod/blob/3463cbbaca81d1d59baf6586979cd91413e2192b/common/src/main/java/org/vivecraft/client_vr/gameplay/trackers/HorseTracker.java#L30-L32

kevina commented 4 months ago

That was not my experience when ridding a horse with 1.20. I need to double check with the latest and compare to 1.12 and get back to you.

fayer3 commented 4 months ago

horse rotations is same as your in room rotation iirc

kevina commented 4 months ago

I did some testing and can confirm that controlling the horse still rotates as if it had reins. At least with my setup with index controllers. The thumb stick is configured to control movement, and the movement direction being controlled by where the left controller is pointing. To verify there was nothing strange about my setup I did this using a clean install in a new folder.

To test this, mount a horse in 1.20, hold your left arm forward than move your right arm, the horse will rotate. Now press forward, the horse will move the direction it is pointing, not the direction the left controller is pointing.

I also tested in 1.12.2-jrbudda-11-r2 and interestingly (at least with the default settings) the horse will just move in the direction you are pointing in-spite of the fact the reins are controlling the direction the horse is pointing.

fayer3 commented 4 months ago

trying it out, I think that is just the a side effect, of how the player model forward rotaion is done, and the fore points where the player model is pointing at

kevina commented 4 months ago

On closer inspection I think you are right as rotating my head without moving my hands changes the direction slightly.

I think the real bug is that the horse moves the direction it is pointing, not the direction your controller is pointing. I will open a new bug report for that.

fayer3 commented 4 months ago

it moves the direction you point at though

kevina commented 4 months ago

Not for me. I just double checked, although it's not always in the direction the horse is pointing.

fayer3 commented 4 months ago

are you playing with controller freemove mode, or hmd?

kevina commented 4 months ago

controller freemove

fayer3 commented 4 months ago

hm, it's using the main hand for some reason

kevina commented 4 months ago

yeah your right

fayer3 commented 4 months ago

no it's not. the horse tracker is non functional atm, it's just the same as regular movement, point an walk

https://github.com/Vivecraft/VivecraftMod/blob/3463cbbaca81d1d59baf6586979cd91413e2192b/common/src/main/java/org/vivecraft/client_vr/gameplay/trackers/HorseTracker.java#L30-L32

also I double checked that on 1.12, the rein horse interaction isn't active there either, when active, it would allow to pull and snap the reins to get hte horse into a auto walk mode

kevina commented 4 months ago

sorry for the noise then, assuming #261 gets fixed, I consider this issue a very low priority