PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Full DPS unable to account for main minion skill and Summon Phantasm support at the same time in one gem link

Open
#2,893 1 comment 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

bug
Dominant language
Lua
Stars
5.4k
Forks
2.4k
Avg merge
1d 12h
Merged PRs (30d)
26

Description

Example pastebin: https://pastebin.com/V3UUp1Nq

Having any minion skill linked to summon phantasm support will prevent Full DPS from accounting for both setups at the same time, not an issue for non-minion/count skills such as puncture + sps

In a similar way that Elemental Hit dps can only display the dps for fire, cold OR lightning, for a Raise zombie linked to Summon Phantasm, the skill setup is only able to select either the phantasms OR the zombies as Include in full dps

A workaround is to duplicate the gem links and have one setup as zombie and the other as phantasm

Related already raised issues:

  1. SPS will inherit skill count, level and quality from main minion skill gem instead of itself
  2. Skill count for abilities such as cobra lash will count the number of cobra lash projectiles as the number of phantasms

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the linked Pastebin example and compare the two related issues about Summon Phantasm skill counts, levels, and quality. Trace how Full DPS handles a minion skill linked to Summon Phantasm; done means both the main minion and phantasm setups can contribute to Full DPS without duplicating the gem links.

Written by the indexing model from the issue text.

Assessment

Tech stack
lua
Domain
game-dev
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.