PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Add (possibly optionally) ascendancy notables to power report

Aperta
#6,274 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

enhancement
Lingua principale
Lua
Stelle
5.4k
Fork
2.4k
Merge medio
1g 12h
PR unite (30g)
26

Descrizione

Check for duplicates
  • I've checked for duplicate issues by using the search function of the issue tracker
Is your feature request related to a problem?

The ascendancy notables do not show in the node power report. I was looking for the highest dps one when searching for the forbidden flame/flesh jewels.

Describe the solution you'd like

https://github.com/PathOfBuildingCommunity/PathOfBuilding/blob/19818870af754cc17b9fa8e16c026670f20932e6/src/Classes/TreeTab.lua#L666C59-L666C59

I noticed the ascendancy notables were missing from the power report when searching for forbidden flesh/flame jewels. I don't know if there is a reason why these are omitted from the power report but it was just a 1 line fix to include them in TreeTab.lua:

            if (node.type == "Normal" or node.type == "Keystone" or node.type == "Notable") and not node.ascendancyName then

can be changed to

            if (node.type == "Normal" or node.type == "Keystone" or node.type == "Notable") then

I don't really know what else this impacts but it worked to include them in the power report output at least. If there is a good reason to exclude them please let me know.

Describe alternatives you've considered

the current alternative is mousing over all of them and checking them independently

Additional context

Let me know if you want to make a PR for this and I will happily do that. I tested it on the latest dev branch and it seemed to be working as expected in a quick spot check but I did not run the tests...

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Direzione di ricerca

Inizia in src/Classes/TreeTab.lua intorno alla riga 666 e controlla come vengono selezionati i nodi per il report del potere. Riproduci la ricerca dei gioielli Forbidden Flame/Flesh, quindi verifica che gli ascendancy notable compaiano nel report senza modificare i risultati degli altri nodi; l’issue indica che i test non sono stati eseguiti.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
lua
Ambito
desktop
Tipo di issue
Funzionalità
Difficoltà
2/5
Tempo stimato
1-3 ore
Stato di attività
Ferma
Chiarezza
Specificata chiaramente
Idoneità per principianti
45/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.