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

removes randomspread #3095

Closed

Conversation

FalloutFalcon
Copy link
Member

About The Pull Request

removes random spread from ammo, guns, and mechs

Why It's Good For The Game

cruft var that either had no effect, guns, or didn't matter since there was already random spread being applied from another source, mechs

Changelog

🆑
del: Removed random spread var
/:cl:

@github-actions github-actions bot added the Code change Watch something violently break. label Jun 10, 2024
@FalloutFalcon FalloutFalcon marked this pull request as ready for review June 10, 2024 13:15
@Kapu1178
Copy link
Contributor

Kapu1178 commented Jun 10, 2024

Dunno why you're removing this when it's useful and actually works. Really stretching the definition of "Cruft". "Random spread" means that the spread per bullet will be completely random within a given range of spread. "Smart spread" (randomspread 0) means that each bullet will be evenly spaced to fill the given range in a fixed pattern.

@FalloutFalcon
Copy link
Member Author

Dunno why you're removing this when it's useful and actually works. Really stretching the definition of "Cruft". "Random spread" means that the spread per bullet will be completely random within a given range of spread. "Smart spread" (randomspread 0) means that each bullet will be evenly spaced to fill the given range in a fixed pattern.

I only tested it with single shot weapons.. looking at the code I understand what you mean idk what I was doing in this pr I think I jumped the gun.

@FalloutFalcon FalloutFalcon deleted the random-spread branch January 7, 2025 03:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code change Watch something violently break.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants