Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Crouch Jitter #480

Merged
merged 1 commit into from
Jul 6, 2024
Merged

Crouch Jitter #480

merged 1 commit into from
Jul 6, 2024

Conversation

AdamTadeusz
Copy link
Contributor

@AdamTadeusz AdamTadeusz commented Jul 5, 2024 •
edited
Loading

Description

Remove jitter when crouch walking and when crouch walking in a tight space

Toolchain

  • Windows MSVC VS2022

Linked Issues

@AdamTadeusz AdamTadeusz requested a review from a team July 5, 2024 16:43
This was referenced Jul 5, 2024
Closed
Closed
@Rainyan
Copy link
Collaborator

Rainyan commented Jul 5, 2024 •
edited
Loading

This fixes the pred error spam for ducked movement, but we're still getting a mismatch for cl_showerror 2 at location setpos 1382.033203 1579.425659 183.281250;setang 13.991999 -3.431316 0.0 of nt_dimensions_test map, trying to duck under the panel between the pre-placed translucent ducked assault & recon models when you are an assault. This would probably be worth fixing here as well, and making sure the recon/support don't have a similar off-by-one-unit discrepancy between cli/srv - either a gap should be able to be fit under, or not, but without jitter either way.

@Rainyan
Copy link
Collaborator

Rainyan commented Jul 5, 2024

Or I guess the duck hull mismatch is handled in #470 ?

@AdamTadeusz
Copy link
Contributor Author

Or I guess the duck hull mismatch is handled in #470 ?

Yes

@AdamTadeusz AdamTadeusz merged commit 58c1fbc into NeotokyoRebuild:master Jul 6, 2024
@AdamTadeusz AdamTadeusz deleted the 004_crouchedMovementJitter branch August 4, 2024 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Rainyan Rainyan Rainyan approved these changes

@nullsystem nullsystem nullsystem approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Crouched movement jittery
3 participants
@AdamTadeusz @Rainyan @nullsystem