Opened 9 years ago

Closed 8 years ago

#1395 closed change (fixed (in v4))

Allow ego types to affect item weight

Reported by: magnate Owned by: magnate
Milestone: v4 Keywords: items,affixes
Cc:

Description

http://angband.oook.cz/forum/showthread.php?p=51066#post51066

Apparently the weight field of an ego type is not currently used.

Change History (7)

comment:1 Changed 9 years ago by Derakon

Given that egos can be applied to different types of items with wildly varying base weights, it's not immediately obvious what to do with the weight field in this situation. I see two possible approaches that make the most sense to me. The first has the weight field specify an additive modifier to weight, so 10 means the item weighs 1 more pound, -10 means it weighs 1 less pound (down to a minimum weight of 0.0 pounds), etc. The second would have it as a percentage multiplier: 10 means the item weighs 10% more, -10 means it weighs 10% less (-100 means it weighs -100% less, i.e. 0.0 pounds).

In either case we wouldn't have to adjust the 0s used elsewhere for egos. I'm inclined to go with the latter solution over the former, and then do something like make dwarven armor weigh a bit more than usual and lighten elvenkind armor.

comment:2 Changed 9 years ago by magnate

  • Owner set to magnate
  • Status changed from new to assigned

Agreed. I prefer the percentage approach too.

comment:3 Changed 8 years ago by magnate

  • Milestone changed from 3.3.0 to Future
  • Status changed from assigned to confirmed

Punting in accordance with new milestone policy (that any other milestone is only set once someone is actually working on the ticket).

comment:4 Changed 8 years ago by magnate

  • Keywords items,affixes added; items removed

This ticket is addressed in (or enabled by) https://github.com/angband/angband/pull/94

comment:5 Changed 8 years ago by magnate

  • Milestone changed from Future to 3.4.0
  • Status changed from confirmed to pending

comment:6 Changed 8 years ago by magnate

  • Milestone changed from 3.4.0 to 4.0

Initial assignment to v4 per http://trac.rephial.org/roadmap

comment:7 Changed 8 years ago by magnate

  • Resolution set to fixed (in v4)
  • Status changed from pending to closed

Fixed in the affixes merge [r48ccc16f]

Note: See TracTickets for help on using tickets.