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

turret/vehicle ammo fix based on research #36083

Merged
merged 12 commits into from
Dec 20, 2019

Conversation

misterprimus
Copy link
Contributor

Kill and loot a turret/robot.

Summary

SUMMARY: Balance "Make robot/turret ammo match research."

Purpose of change

To allow chickenwalker/tank to drop ammo and to match to research how much ammo the automated vehicles and turrets would carry.

Describe the solution

N/A - not really a feature, which was implemented in an earlier PR. This is just JSON adjustments.

I looked up how much these carried, but only came up with the M240 weapon, which is stated to carry 450 rounds of 7.62x51. I found the weight of all the cartridges used by the robots in the game and made the TOTAL ammo weight to be about the same (rounding the ammo qty to an even number).
image

Describe alternatives you've considered

Testing

Spawned and destroyed chickenwalker.

Additional context

Got rid of M16/M4 and replaced with M249 to prevent spawning of mags and make it look like the gun was being operated using a belt, which appears to be the way they are operated IRL.

Copy link
Member

@anothersimulacrum anothersimulacrum left a comment

Choose a reason for hiding this comment

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

Please revert your changes to the CROWS turrets, they are not accurate.

Large ammunition boxes enable for sustained firing periods, carrying 96 rounds for the Mk 19, 400 rounds for the M2, 1,000 rounds for the M240B, and 1,600 rounds for the M249

Wikipedia

misterprimus and others added 2 commits December 14, 2019 00:16
Co-Authored-By: Anton Burmistrov <Night_Pryanik@mail.ru>
Co-Authored-By: Anton Burmistrov <Night_Pryanik@mail.ru>
Copy link
Member

@ymber ymber left a comment

Choose a reason for hiding this comment

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

mon_secubot having an M16A4 is completely deliberate. Add a new mon_talon_m249 if you want but don't touch mon_secubot.

Chicken walkers are on the way to being obsolete. See #35821.

@misterprimus
Copy link
Contributor Author

misterprimus commented Dec 14, 2019

mon_secubot having an M16A4 is completely deliberate. Add a new mon_talon_m249 if you want but don't touch mon_secubot.

Chicken walkers are on the way to being obsolete. See #35821.

The reason I did that was to avoid having the TALON drop STANAG mags. How would that thing reload its gun? Pretty sure they are belt-fed, right? Should I add an box mag to the M16?
https://dealernfa.com/fightlite-mcr060/

@ymber
Copy link
Member

ymber commented Dec 14, 2019

The robot can't reload on its own. The operator has to reload it which is why belt fed weapons are much more common on UGVs. I gave this one an M16A4 because it's much more likely that the places these spawn would have M16s than M249s and they're not for applications where you're likely to need more than 30 rounds.

@misterprimus
Copy link
Contributor Author

misterprimus commented Dec 14, 2019

The robot can't reload on its own. The operator has to reload it which is why belt fed weapons are much more common on UGVs. I gave this one an M16A4 because it's much more likely that the places these spawn would have M16s than M249s and they're not for applications where you're likely to need more than 30 rounds.

Has there ever been a 30-round mag-fed SWORDs unit IRL? I read they can use M16 rifles, but with a 30 round mag? Just doesn't follow the script if they are giving it hundreds of rounds of a bigger cartridge.

@curstwist curstwist added [JSON] Changes (can be) made in JSON Game: Balance Balancing of (existing) in-game features. Ranged Ranged (firearms, bows, crossbows, throwing), balance, tactics Vehicles Vehicles, parts, mechanics & interactions labels Dec 14, 2019
@tenmillimaster
Copy link
Member

The robot can't reload on its own. The operator has to reload it which is why belt fed weapons are much more common on UGVs. I gave this one an M16A4 because it's much more likely that the places these spawn would have M16s than M249s and they're not for applications where you're likely to need more than 30 rounds.

Has there ever been a 30-round mag-fed SWORDs unit IRL? I read they can use M16 rifles, but with a 30 round mag? Just doesn't follow the script if they are giving it hundreds of rounds of a bigger cartridge.

Yes, they're not intended to function without an operator.

@misterprimus
Copy link
Contributor Author

The robot can't reload on its own. The operator has to reload it which is why belt fed weapons are much more common on UGVs. I gave this one an M16A4 because it's much more likely that the places these spawn would have M16s than M249s and they're not for applications where you're likely to need more than 30 rounds.

Has there ever been a 30-round mag-fed SWORDs unit IRL? I read they can use M16 rifles, but with a 30 round mag? Just doesn't follow the script if they are giving it hundreds of rounds of a bigger cartridge.

Yes, they're not intended to function without an operator.

Seems like even a bank secubot would be expected to function without an operator during a shootout situation, though. And it would likely require potentially more than 30 rounds if they think security is a big enough risk to get armed guards. Labs would be even more dangerous since secubots there would be intended to deal with zed groups and high level monsters breaking out.

@misterprimus misterprimus requested a review from ymber December 14, 2019 19:57
@misterprimus
Copy link
Contributor Author

misterprimus commented Dec 14, 2019

The robot can't reload on its own. The operator has to reload it which is why belt fed weapons are much more common on UGVs. I gave this one an M16A4 because it's much more likely that the places these spawn would have M16s than M249s and they're not for applications where you're likely to need more than 30 rounds.

Anyway I guess I'll just leave it as is with the m16 w/ 30 rounds, though it doesn't make sense to me in many of the settings (a big box mag would make more sense).

Copy link
Member

@ymber ymber left a comment

Choose a reason for hiding this comment

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

The remaining changes in this look alright.

@misterprimus
Copy link
Contributor Author

The remaining changes in this look alright.

Yes maybe I'll add a proper "military grade" TALON (with M249) in another PR.

@ymber
Copy link
Member

ymber commented Dec 19, 2019

This hasn't changed since I last reviewed it. I don't think it's particularly useful given that chicken walkers and tank drones are on the way out but I don't specifically object to anything in here.

@misterprimus
Copy link
Contributor Author

This hasn't changed since I last reviewed it. I don't think it's particularly useful given that chicken walkers and tank drones are on the way out but I don't specifically object to anything in here.

I clicked the button since your last post indicated you were OK with the changes, but the reviewer changes requested was still up. Thought I needed to click that to move things along.

While not exactly shockingly useful, it will make the walker be consistent with the beagle and other robots and seems more realistic. That PR you linked says "Removes chicken walkers and beagle minitanks from military roadblocks and from any spawn lists (edit to clarify, they are still specifically called up in some labs and in the necropolis)." No clue what the final verdict is or will be with that PR or when (if ever) it will be merged. No point in having a realism/consistency hole in the game just because we're waiting for a PR that might not get merged and which doesn't even necessarily conflict with this change.

misterprimus and others added 3 commits December 20, 2019 01:38
Co-Authored-By: ZhilkinSerg <ZhilkinSerg@users.noreply.github.com>
Co-Authored-By: ZhilkinSerg <ZhilkinSerg@users.noreply.github.com>
Co-Authored-By: ZhilkinSerg <ZhilkinSerg@users.noreply.github.com>
@ZhilkinSerg ZhilkinSerg merged commit eaafc3b into CleverRaven:master Dec 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Game: Balance Balancing of (existing) in-game features. [JSON] Changes (can be) made in JSON Ranged Ranged (firearms, bows, crossbows, throwing), balance, tactics Vehicles Vehicles, parts, mechanics & interactions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants