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

Guns with Interchangeable Barrel-Assemblies don't correctly spawn, s_gun_1 ammo vending machines selling snacks/drinks, stop adding/removing guns based heavily off Gunbroker results. #74735

Closed
KrunkySmokin opened this issue Jun 23, 2024 · 9 comments
Labels
(S1 - Need confirmation) Report waiting on confirmation of reproducibility

Comments

@KrunkySmokin
Copy link

KrunkySmokin commented Jun 23, 2024

Describe the bug

The bugs:
-Bug 1
Guns that spawn in Gun Stores that in the description of said gun say that come with interchangeable barrel-assemblies do not spawn correctly.
image
image
They are entirely broken, and other guns with similar 'swappable' assemblies are broken in a similar way. The CZ Trail Carbine, the Accuracy International AXMC, and MDRX rifles are all broken and have been for weeks.
-Bug 2
s_gun_1 gun store vending machines spawn food and drinks instead of ammunition.
image
image

--assorted related to third part of title--

Originally posted by @Maleclypse in #74636 (comment) Instead of getting angry at me and telling me to redirect conversation when no one directed it at me or told me before and threaten to remove my ability to comment on a repository, fix the gun bugs your supported Pull Requests create that fester and sit around for months instead of removing guns where a perfectly viable spawn for them exists.

Not angry, just letting you know that you need to read the comments directed at you.
#74636 (comment)

Originally posted by @Maleclypse in #74720 (comment)

It wasn't directed at me as it was said to someone else but alright, because you said someone else replying to someone else saying maybe PRs aren't the best place to bring up grievances was directed at me, it was directed at me and I should take their passive aggressive maybe replying to someone that wasn't me as a warning to myself.

#74720 (comment)

Gun Broker isn't the only determinate factor as to whether or not a gun is popular in New England and should be used in conjunction with multiple other factors such as if the firearm is still currently in production or being imported for sale by a big name company, like Beretta does the MR73. Just because Gun Broker says 15-30 results for you doesn't mean that the gun wouldn't show up in a private gun safe collection or among collectors at a gun show.
The gun show as a spawnable exists and it has over time spawned the more rare guns, and sure one unique/rare gun for a quest might've made it's way there but that's the problem if you're going to claim other rare guns wouldn't be able to be found there. Collectors go around to gun shows, they don't have to be specifically local. There's already a Generic Guns mod and if you want to genericize all the firearms, just take up stewardship of that instead of removing collector firearms from the game.
-----

Attach save file

n/a

Steps to reproduce

Bug 1

  1. Enter gun store and find MDRX, CZ Trail Carbine, or find an AXMC Rifle
  2. Guns do not spawn correctly and cannot retain attachments or magazines

Bug 2

  1. Enter s_gun_1
  2. Use vending machines in back room
  3. Food and drinks for sale instead of ammunition.

Expected behavior

CZ Trail Rifle, Accuracy International AXMC, and MDRX rifles do not spawn correctly, cannot accept attachments, or magazines.
s_gun_1 vending machines don't sell ammunition, only snacks/drinks.

Screenshots

No response

Versions and configuration

OS: Windows 10
Game Version: cdda-experimental-2024-06-23-1253
graphics version: tiles
Language: English
Mods loaded: Dark Days Ahead

Additional context

No response

@KrunkySmokin KrunkySmokin added the (S1 - Need confirmation) Report waiting on confirmation of reproducibility label Jun 23, 2024
@XygenSS
Copy link
Contributor

XygenSS commented Jun 23, 2024

Please make a seperate issue per each bug so that they can be individually addressed, discussed and closed/resolved when fixed

@XygenSS
Copy link
Contributor

XygenSS commented Jun 23, 2024

The second "bug" with ammo vending machines is intended behavior as per PR #72597. The fact that it removed ammo from vending machine gun stores is discussed at Issue #74420.

(yes we know that ammo vending machines exist irl. No it wasn't a "realism" patch, yall can put down the pitchforks.)

@KrunkySmokin
Copy link
Author

The second "bug" is intended behavior as per PR #72597. The fact that it removed ammo from vending machine gun stores is discussed at Issue #74420.

Remove the vending machines from that s_gun_1 then because they make no sense being where they are locked away from the public that would be using them.

@XygenSS
Copy link
Contributor

XygenSS commented Jun 23, 2024

The second "bug" is intended behavior as per PR #72597. The fact that it removed ammo from vending machine gun stores is discussed at Issue #74420.

Remove the vending machines from that s_gun_1 then because they make no sense being where they are locked away from the public that would be using them.

Please comment this on the linked issue as this issue is effectively a duplicate concerning this "bug."

@KrunkySmokin
Copy link
Author

KrunkySmokin commented Jun 23, 2024

The second "bug" is intended behavior as per PR #72597. The fact that it removed ammo from vending machine gun stores is discussed at Issue #74420.

Remove the vending machines from that s_gun_1 then because they make no sense being where they are locked away from the public that would be using them.

Please comment this on the linked issue as this issue is effectively a duplicate concerning this "bug."

yeah I was told above that already completed pull requests are not the place to be talking about the issues so this is a new bug/outstanding issue.

@XygenSS
Copy link
Contributor

XygenSS commented Jun 23, 2024

The second "bug" is intended behavior as per PR #72597. The fact that it removed ammo from vending machine gun stores is discussed at Issue #74420.

Remove the vending machines from that s_gun_1 then because they make no sense being where they are locked away from the public that would be using them.

Please comment this on the linked issue as this issue is effectively a duplicate concerning this "bug."

yeah I was told above that already completed pull requests are not the place to be talking about the issues so this is a new bug/outstanding issue.

Not the PR, the issue. #74420. Which is open

@KrunkySmokin
Copy link
Author

KrunkySmokin commented Jun 23, 2024

The second "bug" is intended behavior as per PR #72597. The fact that it removed ammo from vending machine gun stores is discussed at Issue #74420.

Remove the vending machines from that s_gun_1 then because they make no sense being where they are locked away from the public that would be using them.

Please comment this on the linked issue as this issue is effectively a duplicate concerning this "bug."

yeah I was told above that already completed pull requests are not the place to be talking about the issues so this is a new bug/outstanding issue.

Not the PR, the issue. #74420. Which is open

Instead of commenting on that issue you've known about longer than me and bringing it to the attention of the devs so they can close an issue they already addressed, you instead try to get me to comment on it for you?
Comment yourself if it's a duplicate of an already existing, fixed issue 2 weeks later no one has addressed and it's still open.

@XygenSS
Copy link
Contributor

XygenSS commented Jun 23, 2024

Because I have no particular investment in the issue and I'm not gonna pretend to know what you wish to be addressed by commenting in behalf of you?
Issues being open for a long time isn't indicative of anything in particular, everyone working on the game are volunteers who work on whatever they want to work on.
Also I haven't known of the issue before today, I just happen to remember the vending machine PR in passing and discovered the linked issue while linking you the PR.

@kevingranade
Copy link
Member

Again, one topic per issue, issues with multiple topics will be closed because there is no good way to manage them.

Regarding the gun broker criticisms specifically. We have evaluated a number of options and a threshold of completed sales on gunbroker is the least bad option that has come up. If you do not have a better objective threshold, please do not bring it up again, if you aren't providing a viable alternative you're just heckling.

@kevingranade kevingranade closed this as not planned Won't fix, can't repro, duplicate, stale Jun 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
(S1 - Need confirmation) Report waiting on confirmation of reproducibility
Projects
None yet
Development

No branches or pull requests

3 participants