Skip to content

Mass Contests Affect Camera Shake#560

Merged
DangerRevolution merged 2 commits intoSimple-Station:masterfrom
VMSolidus:Mass-Contests-Camera-Shake
Jul 21, 2024
Merged

Mass Contests Affect Camera Shake#560
DangerRevolution merged 2 commits intoSimple-Station:masterfrom
VMSolidus:Mass-Contests-Camera-Shake

Conversation

@VMSolidus
Copy link
Copy Markdown
Member

Description

I made this entirely on a whim to prove a point in the middle of a conversation. I have not tested this code, but I'm confident that it will work as advertised.

This refactors the public KickCamera function so that it no longer: Divides by 1, divides by 1 again, multiplies by 1, normalizes a vector(turning its A value to 1), and then divides said vector by 1 again for good measure.

Instead 1 has been replaced by MassContest, wooh! I also got rid of redundant steps. The effect of this is that characters with greater than human-standard mass are slightly more resistant to all sources of camera shake(Explosions, gravity generator turning off, gunfire, being hit by flying objects, etc), while characters with less than standard human mass experience a greater amount of camera shake.

You're welcome. Now go find a Felinid to shake vigorously.

Changelog

🆑

  • add: Camera Shake is now affected by character mass. Small characters experience more camera shake. Large characters experience less.

@github-actions github-actions bot added the Changes: C# Changes any cs files label Jul 18, 2024
Copy link
Copy Markdown
Member

@DEATHB4DEFEAT DEATHB4DEFEAT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should test this

@DangerRevolution DangerRevolution merged commit 25056a9 into Simple-Station:master Jul 21, 2024
SimpleStation14 added a commit that referenced this pull request Jul 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changes: C# Changes any cs files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants