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

dangerous pickup #30187

Closed
communistkiro opened this issue May 2, 2019 · 5 comments · Fixed by #37871
Closed

dangerous pickup #30187

communistkiro opened this issue May 2, 2019 · 5 comments · Fixed by #37871
Labels
<Bug> This needs to be fixed Inventory / AIM / Zones Inventory, Advanced Inventory Management or Zones (S2 - Confirmed) Bug that's been confirmed to exist

Comments

@communistkiro
Copy link

communistkiro commented May 2, 2019

Describe the bug
Having the Dangerous pickups setting in the options enabled should enable auto-pick of items passing the filter despite increasing stamina cost (and perhaps various action movement costs?) for you. This doesn't work for auto-foraged items, which are left on the ground.

To Reproduce
Steps to reproduce the behavior:

  1. Have Dangerous pickups: enabled;
  2. Turn on any form of auto-forage;
  3. Reach carry weight limit;
  4. All new to-be-auto-picked-up items remain on tile your char's standing on with the log messages "You harvest and drop %s x%d" and "You see <thing you harvested> x %d here"

Expected behavior
If dangerous pickups is enabled, items that were auto-foraged should be moved to inventory if there is more sufficient volume rather than dropped.

Versions and configuration(please complete the following information):

  • Game Version: 0.D-2518-g65a0506
  • Graphics version: tiles
@MrHrulgin
Copy link
Contributor

MrHrulgin commented May 2, 2019

C:DDA doesn't have a soft weight limit with increasing encumbrance, it's a hard limit. There aren't any other circumstances where you can carry more than your limit; why should auto-forage circumvent that?

EDIT: This is only true for volume, as was pointed out below.

@communistkiro
Copy link
Author

communistkiro commented May 2, 2019

There are two fractions, carry weight and volume. Exceeding the latter results in items dropping, exceeding the former makes make actions cost more movement points. Both indicated by red.

What I meant with 'overencumbered' is that message one gets when carrying over the carry soft limit, not the encumbrance mechanic. Having reached it, you can carry more foraging results at the cost of faster stamina loss, but you have to pick it up manually.

@MrHrulgin
Copy link
Contributor

You're right, my brain completely slipped a cog. I would imagine that it was put in place to keep people from accidentally overloading themselves, but an option to extensively burden yourself makes sense.

@ifreund ifreund added (S1 - Need confirmation) Report waiting on confirmation of reproducibility <Bug> This needs to be fixed Inventory / AIM / Zones Inventory, Advanced Inventory Management or Zones labels May 3, 2019
@ifreund
Copy link
Contributor

ifreund commented May 3, 2019

If dangerous pickups is disabled, items that were auto-foraged should be moved to inventory, rather than dropped.

I think you may have this backwards: enabling not disabling dangerous pickups should cause the behavior you described.

@communistkiro
Copy link
Author

@ifreund , my mistake, I'll edit it.

@ifreund ifreund added (S2 - Confirmed) Bug that's been confirmed to exist and removed (S1 - Need confirmation) Report waiting on confirmation of reproducibility labels May 3, 2019
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 Inventory / AIM / Zones Inventory, Advanced Inventory Management or Zones (S2 - Confirmed) Bug that's been confirmed to exist
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants