Drillneck: "for each time they have Pierced" modifiers are not parsed
I maintainer di solito rispondono entro 4 giorni
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 80/100
Direzione di ricerca
Aggiungi la chiave non contratta accanto alla voce contratta esistente in src/Modules/ModParser.lua:1681, usando lo stesso tag { type = "PerStat", stat = "PiercedCount" }; le stringhe dei dati di gioco UniqueProjectileIncreasedDamagePerPierce1 e UniqueProjectileIncreasedCriticalHitChancePerPierce1 in src/Data/ModItemExclusive.lua mostrano la dicitura esatta da far corrispondere. Poi rigenera src/Data/ModCache.lua in modo che il suffisso residuo " for each time they have Pierced " scompaia dalle righe di Drillneck. Verifica equipaggiando Drillneck, impostando "# of times Skill has Pierced" in Configuration e confermando che DPS e probabilità di critico cambiano; la riga correlata "Attack Projectiles Return if they Pierced" in ModCache.lua:6616 è fuori ambito.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Drillneck (Penetrating Quiver) has two per-pierce modifiers:
Projectiles deal (42-64)% increased Damage with Hits for each time they have Pierced
Projectiles have (42-64)% increased Critical Hit chance for each time they have Pierced
PoB parses the increase but not the condition, so part of each line is left unparsed and neither line is applied, whatever the pierce count is. The generated cache records the remainder:
src/Data/ModCache.lua:8996
c["Projectiles deal 64% increased Damage with Hits for each time they have Pierced"]={{[1]={flags=1024,keywordFlags=262144,name="Damage",type="INC",value=64}}," for each time they have Pierced "}
The game-data wording is "they have Pierced" (src/Data/ModItemExclusive.lua, UniqueProjectileIncreasedDamagePerPierce1 and UniqueProjectileIncreasedCriticalHitChancePerPierce1). ModParser.lua only has the contracted form:
-- src/Modules/ModParser.lua:1681 (dev bb52d6b)
["for each time they've pierced"] = { tag = { type = "PerStat", stat = "PiercedCount" } },
Proposed fix
Add the uncontracted wording next to it, with the same tag, then regenerate ModCache.lua:
["for each time they have pierced"] = { tag = { type = "PerStat", stat = "PiercedCount" } },
Repro
- Equip Drillneck with a projectile skill.
- In Configuration, set "# of times Skill has Pierced" to 1 or 2.
- Damage and crit chance do not change.
Measured effect
With the two lines reworded to "they've Pierced" (which PoB parses), a Frostbolt build against the boss preset:
| # of times Skill has Pierced | Frostbolt DPS | Crit chance |
|---|---|---|
| 0 | 3,572 | 11.4% |
| 1 | 12,660 | 39.1% |
| 2 | 27,308 | 66.8% |
With the item's own wording, every pierce count reads as the first row.
Not in this report
"Attack Projectiles Return if they Pierced at least (2-4) times" is also unparsed (ModCache.lua:6616). Return is a separate mechanic, so it would be a separate report.
- Lingua principale
- Lua
- Stelle
- 1.9k
- Fork
- 455
- Merge medio
- 6g 6h
- PR unite (30g)
- 25
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di PathOfBuildingCommunity/PathOfBuilding-PoE2
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
PathOfBuildingCommunity/PathOfBuilding-PoE2#2156 · 1 commento ·
I maintainer di solito rispondono entro 4 giorni
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
PathOfBuildingCommunity/PathOfBuilding-PoE2#1857 · 2 commenti · 1 reazione ·
I maintainer di solito rispondono entro 4 giorni
-
Build Planner Path SettingApertaenhancement
Difficoltà 3/5 1-2 giorni Idoneità per principianti 50/100
PathOfBuildingCommunity/PathOfBuilding-PoE2#2558 · 3 commenti ·
I maintainer di solito rispondono entro 4 giorni
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 50/100
PathOfBuildingCommunity/PathOfBuilding-PoE2#2556 ·
I maintainer di solito rispondono entro 4 giorni
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 68/100
PathOfBuildingCommunity/PathOfBuilding-PoE2#2552 ·
I maintainer di solito rispondono entro 4 giorni
Tutte le issue di PathOfBuildingCommunity/PathOfBuilding-PoE2
Issue simili
-
omarchy-menu-keybindings lua bind scan spins at 100% CPU when user config iterates a mocked hl APIAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
omacom/omarchy#14302 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
BUG
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
linuxmint/cinnamon-spices-applets#9091 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
Nevcairiel/GatherMate2#62 ·
-
bug
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100