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

Gun volume audit for .357 SIG + adjust SIG Sauer weights #37670

Merged
merged 3 commits into from Feb 4, 2020
Merged

Gun volume audit for .357 SIG + adjust SIG Sauer weights #37670

merged 3 commits into from Feb 4, 2020

Conversation

ghost
Copy link

@ghost ghost commented Feb 3, 2020

Summary

SUMMARY: Content "Realistic gun volumes for .357 SIG"

Purpose of change

Helps #37374. Also the SIG Sauer weights were slightly off.

Describe the solution

Gun volume as surface area times width.

Describe alternatives you've considered

None.

Testing

Loaded ingame.

Additional context

Sig Sauer P226(using E26R-9-LEGION model, as it's the only DA/SA with 15rnd mag like the ingame description), overall length 20.3cm and width 3.8cm: https://www.sigsauer.com/store/p226-legion-full-size.html
Results: https://imgur.com/a/LehCJzT

Glock 31, overall length 20.4cm and width 3.2cm: https://us.glock.com/en/pistols/g31
Results: https://imgur.com/a/DT51ZrF

SIG Sauer P320(320C-357-BSS is their only .357 SIG version), overall length 18.3cm and width 3.4cm: https://www.sigsauer.com/store/p320-nitron-compact.html
Results: https://imgur.com/a/oRrsw5p

Note: P320's magazine has one higher capacity than real life, which will be fixed in another PR.

@ghost ghost marked this pull request as ready for review February 3, 2020 13:12
@ghost ghost changed the title Gun volume audit for .357 SIG Gun volume audit for .357 SIG + adjust SIG Sauer weights Feb 3, 2020
@ZhilkinSerg ZhilkinSerg added [JSON] Changes (can be) made in JSON Items / Item Actions / Item Qualities Items and how they work and interact <Bugfix> This is a fix for a bug (or closes open issue) labels Feb 4, 2020
@I-am-Erk I-am-Erk merged commit 7c40107 into CleverRaven:master Feb 4, 2020
@ghost ghost deleted the Audit357SIG branch February 5, 2020 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bugfix> This is a fix for a bug (or closes open issue) 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.

3 participants