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.

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

feat: Crouching affective player visibility #5806

Merged
merged 2 commits into from
Dec 14, 2024

Conversation

AlecWhite
Copy link
Contributor

@AlecWhite AlecWhite commented Dec 13, 2024

Checklist

Required

Purpose of change

To try fix the problems described in #5113. I have added an increase to stealth when crouching. Also lowered the minimum visibility you can have via crouching and mutation.

The value used is based on the mutation "Crafty" more or less.
Would also be nice for player size to have a bigger effect on stealthiness but I leave it for another PR.

Describe the solution

Added check for player crouching affecting the total stealthiness rating.

Describe alternatives you've considered

None really.

Testing

Loaded new character, crouched around bunnies and see that I can get closer to bunnies without them warping out of the visibility bubble.

Additional context

This is a starting point to fix #5113.

@github-actions github-actions bot added the src changes related to source code. label Dec 13, 2024
@chaosvolt
Copy link
Member

Balance seems okay to me, will compile-test in a bit.

@RoyalFox2140
Copy link
Collaborator

Compiled, Tested. It does something when I crouch and uncrouch. I just don't know how pronunced the effect is. It DOES lower enemy detection range on this pit bull.
image

@chaosvolt
Copy link
Member

Tested and it seems to work, though the effect isn't very pronounced. Since you're slower it also doesn't really help with approaching skittish wildlife much since if they spot you even once they're already zooming away faster than you can run and losing sight of you doesn't change that.

@chaosvolt chaosvolt merged commit c51a539 into cataclysmbnteam:main Dec 14, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
src changes related to source code.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Passive animals run away from the player too much
3 participants