question-mark

Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Issues with "range" on melee weapons

See original GitHub issue

Issue Description

Melee weapons in equipment currently say "range": { "normal": 5 }, which is iffy in some regards:

  1. Technically melee weapons don’t have range (unless they also have thrown), they are specifically not ranged.
  2. The numerical data currently doesn’t reflect Reach property.

My first thought was to bump this by 5ft on Reach weapons, but after a closer look range isn’t accurate at all. For the record there is also a separate throw_range property for thrown.

As far as I can see, the default reach is a rule, not an aspect of any specific weapon:

A melee weapon is used to attack a target within 5 feet of you

So rules-wise this is strictly binary (does weapon have Reach property), not numeric (weapon has X feet reach).

Issue Analytics

  • State:open
  • Created 3 months ago
  • Comments:17 (10 by maintainers)

github_iconTop GitHub Comments

1reaction
Rarstcommented, Dec 10, 2022

Yeah, as above, there are API convenience considerations. I think it might be best to just bite the duplication and have both derived effective reach property at root (5 or 10 in feet, I vaguely remember magic weapons might extend this? even if there aren’t any like that in SRD) and Reach entry in properties list.

To be clear my concern isn’t making API minimal, my concern is making language of the API consistent with that of the rules. 😃

P.S. against having reach in range because we get full circle to initial problem of melee weapons not having Range. 😃

1reaction
bagelbitscommented, Dec 9, 2022

Oh right. We were just renaming that to range?

{
  "index": "trident",
  "name": "Trident",
  "equipment_categories": [{
    "index": "weapon",
    "name": "Weapon",
    "url": "/api/equipment-categories/weapon"
  }, {
    "index": "martial-melee-weapons",
    "name": "Martial Melee Weapons",
    "url": "/api/equipment-categories/martial-melee-weapons"
  }, {
    "index": "martial-weapons",
    "name": "Martial Weapons",
    "url": "/api/equipment-categories/martial-weapons"
  }, {
    "index": "melee-weapons",
    "name": "Melee Weapons",
    "url": "/api/equipment-categories/melee-weapons"
  }],
  "cost": {
    "quantity": 5,
    "unit": "gp"
  },
  "damage": {
    "damage_dice": "1d6",
    "damage_type": {
      "index": "slashing",
      "name": "Slashing",
      "url": "/api/damage-types/slashing"
    }
  },
  "weight": 4,
  "properties": [
    {
      "index": "thrown",
      "name": "Thrown",
      "url": "/api/weapon-properties/thrown"
    },
    {
      "index": "versatile",
      "name": "Versatile",
      "url": "/api/weapon-properties/versatile"
    }
  ],
  "range": {
    "normal": 20,
    "long": 60
  },
  "two_handed_damage": {
    "damage_dice": "1d8",
    "damage_type": {
      "index": "piercing",
      "name": "Piercing",
      "url": "/api/damage-types/piercing"
    }
  },
  "url": "/api/equipment/trident"
}
Read more comments on GitHub >

github_iconTop Results From Across the Web

Problems with Extended Melee Range | VS Battles Wiki Forum
Now, the main issue involves the melee weapon parts. In "Standard Range," weapon examples are listed as "Knives/daggers, gauntlets, ...
Read more >
Why doesn't melee weapon show range ? : r/Warframe - Reddit
The problem is that the range will be different for each attack, and will potentially vary with stances. A single "range" stat isn't...
Read more >
Why dont they show melee weapons range? - Warframe Forums
False, range is the length of the weapon. A Dakra Prime does not magically change length when you do a slide vs combo...
Read more >
Melee weapons range/reach stat in 40k? - Forum - DakkaDakka
Having recently started an aos army, I've been wondering if melee weapon ranged could, or should, be added to 40k.
Read more >
Now that ranged can not escape melee range
Melee leaves no room for ranged to do damage, which is why ranged ran away in the first place. The time to kill,...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found