-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Simplified ammo naming #79600
base: master
Are you sure you want to change the base?
Simplified ammo naming #79600
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I skimmed through it. This is not a comprehensive review.
I can see some reasoning behind this PR, but overall I don't like a lot of the choices you're making here. Just my personal opinion.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall I find this PR to be kinda vague and contradicting in its goals. I see it's trying to simplify ammo names, but why? What's the goal? Changing 30-06 M2 AP to 30-06 AP doesn't really change anything.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We are using the real names of these rounds and in most cases that is not subject to change.
I stopped reviewing individual entries after the first few because nearly everything I've was seeing is not an ok change.
In broad strokes
- cartridge length is part of the name and should not be removed.
- Designations like CB, LR, AUTO, Cowboy, S&W, and many more are part of the name and should not be removed or replaced.
- Ammunition type designations like FMJ, HEAT, and others are terms of art, not vague descriptors. The specificity is better.
This looks like it does not belong to a base game, so I strongly advise you to create a mod for the changes you desire. |
11ab43b
to
f0f506f
Compare
Spell checker encountered unrecognized words in the in-game text added in this pull request. See below for details. Click to expand
This alert is automatically generated. You can simply disregard if this is inaccurate, or (optionally) you can also add the new words to Hints for adding a new word to the dictionary
|
f0f506f
to
87a2c45
Compare
87a2c45
to
07224c3
Compare
07224c3
to
477ffa4
Compare
477ffa4
to
f244244
Compare
Summary
Interface "Simplify ammo names"
Purpose of change
What is the difference between 7.62x39mm and 7.62x51mm? What is 66/27.3x278 Diyua multipurpose? Is 9x18mm RG028 better than 9x18mm 57-N-181S? While I'm still working on renaming guns, I'll publish the renaming of ammunition in a more simplified manner, as is already done for guns.
Describe the solution
Some cartridges have had designations removed/changed. Using the example of the cartridges already available, the remaining cartridges, where applicable, were renamed according as FMJ, JHP/JSP, AP, P+ and a few others. Some ammo now have more descriptive names.Ok, at the end of the day PR is basically making sure the ammo, ammo types, and casings match up with each other.
Table of changes
Describe alternatives you've considered
Testing
The game loads successfully. The modified dialog with ammo conversion at the gunsmith also works.
Additional context