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

Missing Containers and items from starting professions all vanilla +shipped mods #41551

Closed
Maleclypse opened this issue Jun 24, 2020 · 6 comments
Labels
Good First Issue This is a good first issue for a new contributor [JSON] Changes (can be) made in JSON Mutations / Traits / Professions/ Hobbies Mutations / Traits / Professions/ Hobbies

Comments

@Maleclypse
Copy link
Member

Maleclypse commented Jun 24, 2020

Describe the bug

  1. Security guard missing M9 Beretta
  2. Police Officer missing USP handgun and baton
  3. Bionic Operator missing USP handgun.
  4. Military Rifleman missing handgun unknown.
  5. Gangster missing Glock 9mm
  6. Archeologist missing .38 special
  7. Military Automatic Rifleman missing handgun unknown
  8. Military Grenadier missing handgun and grenade launcher
  9. Breacher missing handgun and storage error removes sledgehammer
  10. Operator Undercover missing FN-57
  11. K-9 Officer missing USP and baton
  12. Police Detective missing .357 handgun
  13. Wild West Gunslinger missing Colt .45
  14. Military Marksman missing handgun unknown.
  15. Operator Recon missing M1911.
  16. SWAT Officer missing USP
  17. SWAT CQC missing USP.
  18. Police Sniper missing rifle and USP
  19. Riot Control Officer missing USP
  20. Operator Sniper missing M1911
  21. Operator Hacker missing M1911.
  22. Operator CBRN missing M1911
  23. Uplifted Enforcer missing USP
  24. Bionic Sniper missing USP
  25. Bionic Gangster missing Glock 9mm

Steps To Reproduce

  1. Make a player from one of the above professions, have missing equipment or hit storage error.

Expected behavior

Someone needs to add these items in json or otherwise alter professions.

Screenshots

Versions and configuration

  • OS: Windows
    • OS Version: 10.0 1903
  • Game Version: 0.E-3407-g845884a [64-bit]
  • Graphics Version: Tiles
  • Game Language: System language []
  • Mods loaded: [
    Dark Days Ahead [dda],
    Disable NPC Needs [no_npc_food],
    Aftershock [aftershock],
    Blaze Industries [blazeindustries],
    C.R.I.T Expansion Mod [crt_expansion],
    Magiclysm [magiclysm],
    DinoMod [DinoMod],
    Alternative Map Key [alt_map_key],
    Mythical Martial Arts [MMA],
    Fuji's Military Profession Pack [fuji_mpp],
    SpeedyDex [speedydex],
    Stats Through Kills [stats_through_kills],
    Stats Through Skills [StatsThroughSkills]
    ]

Additional context

Hoping this list will help a new contributor with some easy starting points.

@Maleclypse Maleclypse changed the title WIP Missing Containers and items from starting professions all vanilla +shipped mods Missing Containers and items from starting professions all vanilla +shipped mods Jun 24, 2020
@anothersimulacrum anothersimulacrum added Mutations / Traits / Professions/ Hobbies Mutations / Traits / Professions/ Hobbies [JSON] Changes (can be) made in JSON labels Jun 24, 2020
@anothersimulacrum anothersimulacrum added the Good First Issue This is a good first issue for a new contributor label Jun 24, 2020
@ZhilkinSerg
Copy link
Contributor

Bisecting shows issue appeared after #40295.

@wapcaplet
Copy link
Contributor

wapcaplet commented Jun 28, 2020

In some cases, these guns do not spawn with the starting professions because their holster does not have enough max_item_length to fit the gun's longest_side. For instance the Police Detective gets a standard "holster" that can only fit a 25 cm gun, but their S&W 619 gun is 27.8 cm in length. This could be fixed by switching out the holster with a larger one.

In other cases, the gun itself does fit in the holster, but the gun's MAGAZINE_WELL pocket is not "rigid": true, OR the pocket does not have a "magazine_well": "NN cm" specifying the internal space of the well - meaning these guns get larger when a magazine is inserted. For example the Security Guard's Beretta M9A1 is only 0.53 L, which would fit in the holster, but when a magazine is inserted, the gun's total volume becomes 0.85 L, so it no longer fits in the holster, and won't spawn. (This specific problem is described in more detail in #40609). This could be fixed by defining a "magazine_well" volume for the gun, such that it could (for example) hold a 15-round magazine without increasing in total volume, while a 30-round magazine would stick out and add to the total volume.

@meladath
Copy link
Contributor

Hopefully a lot of the the pistols are fixed now that #41476 has been merged. I think the standard holster needs to be increased to 30cm, either that or lots of guns need to be made a bit smaller as right now we have quite a few guns that I dont really consider worthy of only being able to fit in an Xl holster that are >25 and <30 cm.

@Hulkninja
Copy link

Gunslinger still borked. Wouldn't this be circumvented by spawning in with the weapon wielded?

@stale
Copy link

stale bot commented Aug 9, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. Please do not 'bump' or comment on this issue unless you are actively working on it. Stale issues, and stale issues that are closed are still considered.

@stale stale bot added the stale Closed for lack of activity, but still valid. label Aug 9, 2020
@KorGgenT KorGgenT removed the stale Closed for lack of activity, but still valid. label Aug 15, 2020
@wapcaplet
Copy link
Contributor

wapcaplet commented Sep 16, 2020

Spot-checked several of the vanilla professions in 0.E-5926-g a883429 - except for an occasional missing baton, they seem to be working now:

1 - Security guard has M9A1 Beretta:
image

5 - Gangster has Glock 19:
image

6 - Archaeologist has S&W 619:
image

11 - K9 Officer has USP .45, but no baton
image

12 - Police Detective has S&W 619:
image

13 - Wild West Gunslinger has Uberti Cattleman .45 Colt (with extra ammo in the holster):
image

17 - SWAT CQC has USP 9mm and baton:
image

19 - Riot Control Officer has USP .45 and tonfa
image

Does anyone know of any of the other professions still having issues? Between this issue and #41969, it looks like all the starting profession items are spawning as expected now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Good First Issue This is a good first issue for a new contributor [JSON] Changes (can be) made in JSON Mutations / Traits / Professions/ Hobbies Mutations / Traits / Professions/ Hobbies
Projects
None yet
Development

No branches or pull requests

7 participants