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

Changes to rifle rounds #12876

Merged
merged 11 commits into from Jul 17, 2015

Conversation

Projects
None yet
4 participants
@ghost
Copy link

commented Jul 9, 2015

Context: http://smf.cataclysmdda.com/index.php?topic=10693.0

Changes:
Renamed 5N7 to 7N10
Changed damage for all 7.62x39mm rounds, all 5.45x39mm rounds, .223, and .308
Changed descriptions for all 7.62x39mm rounds, all 5.45x39mm rounds, .223, and .308
Minor changes to recoil/accuracy/pierce for all affected rounds

@ghost ghost changed the title Changes to 7.62x39mm, 5.45x39mm, and .308 Changes to 7.62x39mm, 5.45x39mm, .223, and .308 Jul 9, 2015

@chaosvolt

This comment has been minimized.

Copy link
Contributor

commented Jul 9, 2015

Ah, interesting. For once I'm not to blame for any oddities on the 5.45x39mm, as I ported the ammo straight from Icecoon's Weapons Pack.

@kevingranade

This comment has been minimized.

Copy link
Member

commented Jul 9, 2015

@chaosvolt

This comment has been minimized.

Copy link
Contributor

commented Jul 9, 2015

That would be good to have references, yes. As I've found when statting out firearms, that can be hard to come by at times.

@ghost

This comment has been minimized.

Copy link
Author

commented Jul 9, 2015

I'll compile a list of values and relevant information used for each round. Should I include it in the repo or keep it external?

@kevingranade

This comment has been minimized.

Copy link
Member

commented Jul 9, 2015

@ghost

This comment has been minimized.

Copy link
Author

commented Jul 9, 2015

Got it. I'll sort that out tomorrow.

@@ -1598,11 +1598,11 @@
"to_hit" : 0,
"ammo_type" : "308",
"casing" : "308_casing",
"damage" : 36,
"damage" : 60,

This comment has been minimized.

Copy link
@Coolthulhu

Coolthulhu Jul 10, 2015

Contributor

60 is a lot

@ghost

This comment has been minimized.

Copy link
Author

commented Jul 10, 2015

Added AMMO_STATS.txt with all relevant info for this PR. I'll update it for the last one when I find the figures.

Semi-OT: Why are the files in ./doc/ all caps?

@karlnp

This comment has been minimized.

Copy link
Contributor

commented Jul 10, 2015

IIRC documents are written in all-caps historically so they show at the head of a directory listing.

@chaosvolt

This comment has been minimized.

Copy link
Contributor

commented Jul 11, 2015

Are you plotting a tweak to .50 BMG anytime soon? 10 rounds of AP and API take up the same 2 volume that 30 rounds of Ball do, because someone forgot that stack size is a thing. o3o

SpongeTheGlaive SpongeTheGlaive
@ghost

This comment has been minimized.

Copy link
Author

commented Jul 13, 2015

@chaosvolt Fixed.

@chaosvolt

This comment has been minimized.

Copy link
Contributor

commented Jul 13, 2015

...I said volume. >_>

In fact, you made the variants way lighter than the normal ammo. Weight controls weight per bullet, not weight per stack. Volume controls how much volume a stack counts as, count controls how much is considered a stack IF stack_size is undefined, and stack_size is, well...what I was saying should've been added.

EDIT: For reference, regular .50 BMG Ball has a count of 30 and a volume of two, meaning 30 bullets is 2 volume (and 15 bullets is 1 volume). The AP and AP variants have the same volume but a count of 10, meaning that same 30 bullets would be 6 volume instead, because whoever added them in the first place forgot to add "stack_size" : 30 to them.

SpongeTheGlaive added some commits Jul 13, 2015

SpongeTheGlaive SpongeTheGlaive
Revert "Fixes .50BMG weights."
This reverts commit b4e6fff.
@ghost

This comment has been minimized.

Copy link
Author

commented Jul 13, 2015

10 .50 rounds now take one volume instead of 0.6 with stack sizes being accounted for. This is reasonable given the size of the ammunition.

@chaosvolt

This comment has been minimized.

Copy link
Contributor

commented Jul 13, 2015

EDIT: Ah, I see. You upped the volume of the regular version, and lowered the AP versions to match. Nevermind then.

@ghost

This comment has been minimized.

Copy link
Author

commented Jul 13, 2015

All 3 types have the same volume per round now. Stack size is the only difference.

Different bullets can't be significantly larger than standard ball because they would not fit in .50 cases.

@chaosvolt

This comment has been minimized.

Copy link
Contributor

commented Jul 13, 2015

As I see. Sorry about that.

@ghost ghost changed the title Changes to 7.62x39mm, 5.45x39mm, .223, and .308 Changes to rifle rounds Jul 14, 2015

SpongeTheGlaive added some commits Jul 15, 2015

SpongeTheGlaive SpongeTheGlaive
SpongeTheGlaive SpongeTheGlaive
@ghost

This comment has been minimized.

Copy link
Author

commented Jul 15, 2015

That's all the rifle rounds done. This is ready to merge now.

@chaosvolt

This comment has been minimized.

Copy link
Contributor

commented Jul 15, 2015

Ooh, interesting. I do regret that we can't have the tracer properties show up on an "every few shots" basis, so as it is every round fired from the tracer mix has its incendiary properties.

@ghost

This comment has been minimized.

Copy link
Author

commented Jul 15, 2015

If I can work out how ranged.cpp works I might try to implement that. I have the c++ skills of a caveman so I doubt it will be done soon, if ever.

@chaosvolt

This comment has been minimized.

Copy link
Contributor

commented Jul 15, 2015

No idea how to implement such a thing either. I'm struggling with rather basic things like adding vehicle groups to mapgen_functions.cpp.

@karlnp

This comment has been minimized.

Copy link
Contributor

commented Jul 15, 2015

adds it to his pile

@kevingranade kevingranade self-assigned this Jul 17, 2015

@kevingranade kevingranade merged commit 8da3a58 into CleverRaven:master Jul 17, 2015

1 check passed

default
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.