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

Gathering water into wielded container do not change weight #43328

Closed
starytroll opened this issue Aug 30, 2020 · 3 comments · Fixed by #44661
Closed

Gathering water into wielded container do not change weight #43328

starytroll opened this issue Aug 30, 2020 · 3 comments · Fixed by #44661
Labels
<Bug> This needs to be fixed Items: Containers Things that hold other things (S2 - Confirmed) Bug that's been confirmed to exist
Milestone

Comments

@starytroll
Copy link

Describe the bug

It seems that pouring water from a pool to wielded tank does not change weight of carried item, so that I can travel with it without additional pain/stamina cost.

Steps To Reproduce

  1. Wield metal tank.
  2. 'e'xamine water tile and put the water into tank.
  3. Check 'i'nventory and see that total weight remains unchanged.

Expected behavior

The carried mass should change after filling the container.

Screenshots

obraz

Versions and configuration

  • OS: Windows
    • OS Version: MINGW/CYGWIN/MSYS2 on unknown Windows version
  • Game Version: 0.E-5457-gaeb89fd [64-bit]
  • Graphics Version: Tiles
  • Game Language: English [en]
  • Mods loaded: [
    Dark Days Ahead [dda],
    Disable NPC Needs [no_npc_food],
    Alternative Map Key [alt_map_key]
    ]

Additional context

Probably it is the case for other containers as well.

@mqrause
Copy link
Contributor

mqrause commented Aug 30, 2020

Seems like an issue with the weight cache not always being up to date. With a bunch of tries, I can pick up a filled 60l tank without adding any weight to my inventory. The issue solves itself after a while, though.

@anothersimulacrum anothersimulacrum added <Bug> This needs to be fixed Items: Containers Things that hold other things labels Aug 30, 2020
@anothersimulacrum anothersimulacrum added this to the 0.F milestone Aug 30, 2020
@wapcaplet
Copy link
Contributor

wapcaplet commented Sep 4, 2020

I can reproduce this issue in 0.E-5679-g446c750473.

Starting with the 4.0 kg of clothes I'm wearing:

image

Picking up and wielding an empty 60L tank (5.67 kg), my weight is up to 9.7 kg:

image

Going down to the river and filling up the 60L tank completely, I see my weight has not changed, even if I walk around for a bit afterwards:

image

Looking at the tank, it shows its own weight correctly as 65.67 kg:

image

If I drop the tank, then pick it up (and wield it) again, it becomes weightless, and I am back to my original 4.0 kg:

image

However, if I get another empty 60L tank, and fill it up from the toilets in the evac shelter basement:

image

It is not enough units to completely fill the tank, and I see the weight correctly updates in my inventory:

image

Dropping and picking up this partly full (partly empty) tank, my inventory weight is correctly updated.

So it seems this only affects full tanks for some reason.

@wapcaplet wapcaplet added the (S2 - Confirmed) Bug that's been confirmed to exist label Sep 4, 2020
@RoyBerube
Copy link
Contributor

Some testing I've done indicates that if you have enough space to put the item in inventory, then the weight is calculated correctly. It does not apply just to fluid tanks.

For example with a hiking backpack that cannot be held in inventory, wearing it adds no weight. Adding items to it can trigger the weight to be displayed properly but not always. With a car battery and hammer contained and wearing it from the floor, it adds no weight. Unloading the backpack then shows the correct weight.

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: Containers Things that hold other things (S2 - Confirmed) Bug that's been confirmed to exist
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants