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

Clothing items collide that shouldn't #36634

Closed
ymber opened this issue Jan 2, 2020 · 5 comments
Closed

Clothing items collide that shouldn't #36634

ymber opened this issue Jan 2, 2020 · 5 comments
Labels
<Bug> This needs to be fixed Items / Item Actions / Item Qualities Items and how they work and interact (P3 - Medium) Medium (normal) priority

Comments

@ymber
Copy link
Member

ymber commented Jan 2, 2020

Describe the bug

Wearable items that shouldn't collide in clothing layers often do. This is most visible in SKINTIGHT and BELTED.

Steps To Reproduce

Steps to reproduce the behavior:

  1. Wear knee pads and a leg ammo pouch at the same time.

Expected behavior

Clothes that are normally worn over each other do not cause extra encumbrance penalties. Clothes that occupy the same layer but different areas do not cause extra encumbrance penalties.

Screenshots

2020-01-05_1534_02
2020-01-05_1534_29
2020-01-05_1534_59
2020-01-05_1535_17

Versions and configuration

  • OS: N/A
  • Game Version: Current master at fe0e3de
  • Graphics version: N/A
  • Mods loaded: N/A
@ymber ymber added (P3 - Medium) Medium (normal) priority <Bug> This needs to be fixed Items / Item Actions / Item Qualities Items and how they work and interact labels Jan 2, 2020
@wapcaplet
Copy link
Contributor

The examples you provided look like legitimate clothing layer conflicts to me:

  • knee pads and leg ammo pouch are both strapped to the legs
  • boy shorts, compression shorts, and leggings are all worn next to the skin on the legs
  • sports bra and compression top are both worn next to the skin on the torso

In your screenshots, the messages indicate why the items are in conflict - wearing multiple items on the same layer, on the same body part, is adding the extra encumbrance. My understanding of the clothing layer system is that's how it's supposed to work. Have I misunderstood, or am I missing something?

@ymber
Copy link
Member Author

ymber commented Aug 26, 2020

Yeah this is expected behavior for the current system. I want to improve the system so we've got better granularity in body part coverage and layers. Compression shorts are a bad example but we have some things that collide in the same layer that are normally worn together. The knee pads and ammo pouch shouldn't collide because theyre on different parts of the leg. I meant to handle this one before 0f but I got caught up with other things.

@Rivet-the-Zombie
Copy link
Member

Maybe we could add separate segments for thigh, knee, and calf? Same for the arms, and maybe the torso (abdomen, chest, neck, back) so that your scarf wouldn't be trying to occupy the same space as your mouthguard &etc.

@hexagonrecursion
Copy link
Contributor

knee pads and a leg ammo pouch no longer collide

  • OS: Linux
    • OS Version: LSB Version: :core-4.1-amd64:core-4.1-noarch; Distributor ID: Fedora; Description: Fedora release 34 (Thirty Four); Release: 34; Codename: ThirtyFour;
  • Game Version: 0d43a6e [64-bit]
  • Graphics Version: Tiles
  • Game Language: English [en]
  • Mods loaded: [
    Dark Days Ahead [dda],
    Disable NPC Needs [no_npc_food],
    Graphical Overmap [Graphical_Overmap]
    ]

@NetSysFire
Copy link
Member

Can confirm. Looks like it is fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bug> This needs to be fixed Items / Item Actions / Item Qualities Items and how they work and interact (P3 - Medium) Medium (normal) priority
Projects
None yet
Development

No branches or pull requests

5 participants