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

AR pistols #37768

Merged
merged 11 commits into from
Mar 10, 2020
Merged

AR pistols #37768

merged 11 commits into from
Mar 10, 2020

Conversation

RarkGrames
Copy link
Contributor

@RarkGrames RarkGrames commented Feb 6, 2020

Summary

SUMMARY: Content "Added two AR pistols"

Purpose of change

Adding some more compact guns to the game that use 5.56mm ammo, using the suggestions and review from @tenmillimaster.

Describe the solution

Testing

Spawned and fired the guns, checked out their spawn rate in the item groups.

Additional context

Thanks to @tenmillimaster and @laveyanfiend for the volume calculations.

@Brian-Otten
Copy link
Contributor

Adding some more guns, even if rare, is always appreciated by me, and since these use common ammo's they will probably see some use.

"copy-from": "rifle_semi",
"looks_like": "skorpion_61",
"type": "GUN",
"name": { "str": "DB-15" },
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Call it a Pistol AR? Diamondback didn't pioneer this, it just describes a type of AR. Singling it out as the DB-15 is very odd.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd argue that pistol AR is the generic name, and every single gun (outside of generic guns) uses brand names, so using the brand name seems correct.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So we're supposed to add Aero precision brand Ar-15, Radical firearms AR pistols, PSA AR pistols, BCM AR pistols, Daniel Defense AR pistols, Anderson AR Pistols, Doublestar AR pistols, Springfield Armory SAINT, CBC industries AR pistols...

Copy link
Contributor Author

@RarkGrames RarkGrames Feb 6, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the difference between and AR pistol made by Diamondback and a rifle made by Remington? This is just one specific model by one random company. This doesn't have to imply that this is the only or even the best AR pistol on the market.

I'm genuinely curious, because I don't really see what is the difference here. Are there several different versions of the Remington 700 rifle made by X company, as opposed to a Remington 700 rifle made by Y company?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If someone wants to, i suppose you could add those weapons to the game if you wanted. We tend to draw the line at a certain point because people can't be bothered to add more guns of the same type, but switching to generic names here and now seems like a weird place.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There isn't a difference. If you add specifically the diamondback model, then you're looking at a very small marketshare, and this gun would go back to being very rare. Make it general, and they're common.

Copy link
Contributor Author

@RarkGrames RarkGrames Feb 6, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's clear. My research was to simply google the most common 7.5 inch AR pistols on the market, and this was the first one roughly fitting the pictures used in volume calculations you gave earlier.

I also don't think that generalization is a good idea here. I can change the name to match the most common manufacturer, then lower the spawn weights to match the actual number of guns on the market, if you can provide some guesses about that.

Or we can implement the generalized "AR pistol" name if you want that.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I replaced the DB-15 with a generic AR pistol item.

"looks_like": "skorpion_61",
"type": "GUN",
"name": { "str": "OA-93" },
"description": "An AR-15 derivative AR pistol manufactured by Olympic Arms in the nineties. The main difference compared to the AR-15 is that the recoil spring has been moved to the top of the gun, circumventing the necessity for a solid buttstock.",
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"AR-15 derivative AR pistol" is a bit clunky of a phrase

@eilaattwood
Copy link
Contributor

The "barrel_length": "0 ml" line gives an error on load, but the Mossberg 590A1 already in the game has the same value.

It has barrel_length": 0 actually. Int value, not string. Don't know whether this is legacy code(shoundn't be used then) or not.

@RarkGrames
Copy link
Contributor Author

You are right. I changed it to int.

@ifreund ifreund added <Enhancement / Feature> New features, or enhancements on existing [JSON] Changes (can be) made in JSON Ranged Ranged (firearms, bows, crossbows, throwing), balance, tactics 0.E Content Freeze labels Feb 8, 2020
Copy link
Member

@tenmillimaster tenmillimaster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wasn't able to find any carbine length OA93's, doing our standard candle gunbroker stuff, even in completed listings. This suggests that it's too rare to be added; There are plenty of other gas-piston side folding options on the market however.

I suggest focusing your efforts solely on the AR-pistols and the OA-93 pistol, and dropping the carbine.

data/json/itemgroups/guns.json Outdated Show resolved Hide resolved
data/json/itemgroups/guns.json Outdated Show resolved Hide resolved
data/json/itemgroups/guns.json Outdated Show resolved Hide resolved
data/json/items/gun/223.json Outdated Show resolved Hide resolved
data/json/items/gun/223.json Outdated Show resolved Hide resolved
data/json/items/gun/223.json Outdated Show resolved Hide resolved
@ZhilkinSerg ZhilkinSerg changed the base branch from master to dev March 6, 2020 10:40
@ZhilkinSerg ZhilkinSerg changed the base branch from dev to master March 6, 2020 10:41
@ZhilkinSerg ZhilkinSerg changed the base branch from master to dev March 10, 2020 09:09
@ZhilkinSerg ZhilkinSerg merged commit 11af4ec into CleverRaven:dev Mar 10, 2020
ZhilkinSerg pushed a commit that referenced this pull request Mar 10, 2020
ZhilkinSerg pushed a commit that referenced this pull request Mar 15, 2020
@ZhilkinSerg ZhilkinSerg mentioned this pull request Mar 15, 2020
13 tasks
ZhilkinSerg pushed a commit that referenced this pull request Mar 15, 2020
ZhilkinSerg pushed a commit that referenced this pull request Mar 16, 2020
ZhilkinSerg pushed a commit that referenced this pull request Mar 17, 2020
ZhilkinSerg pushed a commit that referenced this pull request Mar 17, 2020
ZhilkinSerg pushed a commit that referenced this pull request Mar 18, 2020
ZhilkinSerg pushed a commit that referenced this pull request Mar 29, 2020
ZhilkinSerg pushed a commit that referenced this pull request Apr 1, 2020
ZhilkinSerg pushed a commit that referenced this pull request Apr 1, 2020
ZhilkinSerg pushed a commit that referenced this pull request Apr 2, 2020
ZhilkinSerg pushed a commit that referenced this pull request Apr 2, 2020
@ghost ghost mentioned this pull request Jun 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Enhancement / Feature> New features, or enhancements on existing [JSON] Changes (can be) made in JSON Ranged Ranged (firearms, bows, crossbows, throwing), balance, tactics
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants