CleverRaven / CleverRaven/Cataclysm-DDA
AIM: wrong calculation of how much of a long item destination can hold
- Dominant language
- C++
- Stars
- 13.2k
- Forks
- 4.6k
- Avg merge
- 2d 1h
- Merged PRs (30d)
- 205
Description
### Describe the bug
I have 29 pointy sticks on a table. I also have some space in my inventory to store those. AIM when 'M'oving claims to be able to put 28 pointy sticks in. De-facto, only 1 pointy stick could be moved to inventory
### Steps to reproduce
Unsure of exact steps/measurements of destination inventory involved.
### Expected behavior
'M'oving calculates amount of items you can hold properly
### Screenshots
### Versions and configuration
These are the outputs for the experimental build of commit 00a0caa64d7838379418150e9e3189840853beef
### Additional context
_No response_
Contributor guide
Research direction
Start by reproducing the reported AIM move case with 29 pointy sticks, where the calculation claims 28 can fit but only 1 moves. Trace the destination inventory capacity calculation and compare it with the actual transfer behavior. Done means the displayed movable amount matches the number of items that can actually be moved.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- game-dev
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100