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

Audit holster capacities #37741

Merged
merged 3 commits into from Feb 6, 2020
Merged

Audit holster capacities #37741

merged 3 commits into from Feb 6, 2020

Conversation

ghost
Copy link

@ghost ghost commented Feb 5, 2020

Summary

SUMMARY: Content "More realistic holster volumes"

Purpose of change

Helps #37374, required for #37681.

Describe the solution

Made table of my already merged audits(plus #37374 and #37561), pistols marketed for (sub)compactness or CC highlighted:
image

Fast draw/deep conceal/ankle holster: Min is derringer/Spectrum rounded down to 100ml, max is the highest of highlighted.

XL Holster: Min based on Skorpion Vz. 61 retracted/if the folding stock didn't exist, Max is Skorpion with Stock rounded up to nearest quarter liter for futureproofing.

Holster, regular: Min based on top 3 of the compact guns, max based on largest non-machine pistol rounded up.

Back holster: Min based on smallest rifle audited so far, max based on largest w/ loaded while rounded up to nearest liter for futureproofing.

Describe alternatives you've considered

Adding Tonkatsu's audit volumes on the LeMat Revolver, shotguns, and Beretta M9A1 to my table as well to decide volumes.

Testing

Loaded ingame. Marking as draft so any professions can have a holster swap before merging.

Additional context

The issue of thinner rifles and thicker handguns fitting in inappropriate holsters should disappear with KorG's holster refactor.

Some guns will indeed not be able to be holstered at all (Remington MCS, for example).

@ghost ghost requested a review from tenmillimaster February 5, 2020 22:07
@ghost
Copy link
Author

ghost commented Feb 5, 2020

Only Travis error is the Uberti Cattleman being too big, which is fixed in #37681. 🎉

@ghost ghost marked this pull request as ready for review February 5, 2020 23:15
@ZhilkinSerg ZhilkinSerg added [JSON] Changes (can be) made in JSON Items / Item Actions / Item Qualities Items and how they work and interact labels Feb 6, 2020
@ZhilkinSerg ZhilkinSerg merged commit 7f9aeeb into CleverRaven:master Feb 6, 2020
@ghost ghost deleted the HolsterAudit branch February 6, 2020 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Items / Item Actions / Item Qualities Items and how they work and interact [JSON] Changes (can be) made in JSON
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants