Skip to content

Speedchanger refinements#1667

Merged
Interkarma merged 9 commits intoInterkarma:masterfrom
petchema:speedchanger-refinements
Feb 2, 2020
Merged

Speedchanger refinements#1667
Interkarma merged 9 commits intoInterkarma:masterfrom
petchema:speedchanger-refinements

Conversation

@petchema
Copy link
Copy Markdown
Collaborator

FIx for #1588, and then some (like lowering player's noise volume when sneaking).

Note:
playerMotor.speed is determined by both world conditions and player's input. While it's only required during FixedUpdate() for modifying movement(*), player's input can change with each Update() so that input must be recorded separately.
*. Except in LevitateMotor, that can adjust movement with each frame. If we care, we could update speed with each Update() too.

Pierre Etchemaite added 9 commits January 15, 2020 23:32
If jumping while running, keep forward momentum when touching ground.

You can only switch between running and non-running while grounded.
Interestingly, while hopping around, switch only happen on 2nd touch to
the ground, not sure why.
Speed is determined by both world conditions and player's input.
While it's only required during FixedUpdate() for modifying movement(*),
player's input can change with each Update() so that input must be
recorded separately.

*. Except in LevitateMotor, that can adjust movement with each frame. If
we can, we could update speed with each Update() too.
Should the horse be clever enough to not neigh so loudly when you're
trying to sneak?
It was too quiet, specially with mods like DREAM.

Volume difference should be noticeable (helping convey you're sneaking),
but footsteps should still be audible, as much as possible.
@Interkarma
Copy link
Copy Markdown
Owner

Works well and no problems found. Thank you. :)

@Interkarma Interkarma merged commit 02ac2e2 into Interkarma:master Feb 2, 2020
@petchema petchema deleted the speedchanger-refinements branch February 9, 2020 07:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants