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

Lower the range of dirt guns spawn with #36341

Merged
merged 1 commit into from
Dec 22, 2019

Conversation

anothersimulacrum
Copy link
Member

Summary

SUMMARY: None

Purpose of change

Very very few guns should be fouled heavily, or fouled at all. However, because the amount of dirt guns spawn with is simply a number between the min an max defined here, having a range this high leads to most guns being fouled or fairly heavily fouled.

Describe the solution

Adjust this range back to what is was previously, which leads to most guns being very lightly fouled or not noticeably fouled. Itemgroups can still override this, and make guns spawn with more fouling.

Testing

Before
image
After
image

I hope the fouling levels on the before guns show why this change is necessary. Most of these guns are very heavily fouled.

Additional context

This is effectively reverting the changes made to this in #34470. I'm sure there was a reason for this change, but I cannot find any commentary on it in the PR.

Very very few guns should be fouled heavily, or fouled at all. However,
because the amount of dirt guns spawn with is simply a number between
the min an max defined here, having a range this high leads to most guns
being fouled or fairly heavily fouled.

Adjust this range back to what is was previously, which leads to most
guns being very lightly fouled or not noticeably fouled.
Itemgroups can still override this, and make guns spawn with more
fouling.
@ZhilkinSerg ZhilkinSerg added [C++] Changes (can be) made in C++. Previously named `Code` Items / Item Actions / Item Qualities Items and how they work and interact Ranged Ranged (firearms, bows, crossbows, throwing), balance, tactics labels Dec 22, 2019
@ZhilkinSerg ZhilkinSerg merged commit 234f3fc into CleverRaven:master Dec 22, 2019
@anothersimulacrum anothersimulacrum deleted the dirt-rng branch December 22, 2019 18:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C++] Changes (can be) made in C++. Previously named `Code` Items / Item Actions / Item Qualities Items and how they work and interact Ranged Ranged (firearms, bows, crossbows, throwing), balance, tactics
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants