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

Kill off ammo vending machines #72597

Merged
merged 1 commit into from
Apr 9, 2024
Merged

Kill off ammo vending machines #72597

merged 1 commit into from
Apr 9, 2024

Conversation

kevingranade
Copy link
Member

Summary

None

Purpose of change

"Ammo vending machines"

Describe the solution

Remove.

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Map / Mapgen Overmap, Mapgen, Map extras, Map display Spawn Creatures, items, vehicles, locations appearing on map Items: Ammo / Guns Ammunition for all kinds of weapons and these weapons themselves astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions labels Mar 24, 2024
@PatrikLundell
Copy link
Contributor

It seems this just removes the ability to get (some of the?) ammo from gun stores, without spawning replacement ammo elsewhere?

I'm fine with the removal of the machines, but the ammo ought to be spawned by other means.

If it's considered too much work to locate all the places where these machines were placed and distribute ammo in them, I'd rather use a cheap replacement of providing some, preferably locked, furniture with ammo in it in the palette.

@harakka
Copy link
Member

harakka commented Mar 25, 2024

Afaik this is happening as part of investigating where and how much ammo currently spawns and fixing it to match reality. The goal isn't to preserve the current numbers.

@PatrikLundell
Copy link
Contributor

If that's a thought out move, then that's fine. It looked like an extension of the vending machine transformation to me, though, which is why I voiced my concern.

@kevingranade
Copy link
Member Author

A cursory glance at the item group shows that it's only spawning ammo that spans in extremely large quantities elsewhere, so there's certainly no risk of this change making anything unavailable.

If you're talking about absolute quantities of ammo spawned, as harakka says that's currently in a state of total nonsense and this is part of an effort to fix it.

This actually predates the recent look at vending machine stuff and i just had it sitting on a local branch for months.

@kevingranade kevingranade force-pushed the kill-off-ammo-vending branch from 5e60c60 to 9d4214c Compare March 25, 2024 21:27
@kevingranade kevingranade force-pushed the kill-off-ammo-vending branch from 9d4214c to bdd5062 Compare March 27, 2024 18:33
@github-actions github-actions bot added Mods Issues related to mods or modding BasicBuildPassed This PR builds correctly, label assigned by github actions labels Mar 27, 2024
@Fris0uman Fris0uman merged commit e602eb3 into master Apr 9, 2024
18 of 24 checks passed
@kevingranade kevingranade deleted the kill-off-ammo-vending branch April 9, 2024 15:23
@Rasengetsu
Copy link

Rasengetsu commented Jun 9, 2024

As suspected by Patrik, this actually did remove all ammo from the gunstore variant that used ammo vending machines since all the ammo spawns of this variant were tied to the vending machines.
I would recommend adding safes that follow the same spawn rules as the other unlooted gunstore variants as it makes no sense to have an unlooted gunstore with many guns and a large amount of empty magazines but no bullets for them. Anyone who would loot the ammo would also take the magazines and in many cases the guns too.

@Fris0uman
Copy link
Contributor

You should open an issue about if it's not done already. Closed PR are not the correct place to track bugs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions Items: Ammo / Guns Ammunition for all kinds of weapons and these weapons themselves [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions Map / Mapgen Overmap, Mapgen, Map extras, Map display Mods Issues related to mods or modding Spawn Creatures, items, vehicles, locations appearing on map
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants