The node should not be granting the 50% block chance to the player if you are using Necromantic Aegis with it
Co-authored-by: LocalIdentity <localidentity2@gmail.com>
The option to show only 1x the players hit DPS was not working for the alternate versions of the skill cause we were looking for a exact match on the name
Also realised that the config option was not even showing up for Dominating Blow of inspiring so fixed that too
Co-authored-by: LocalIdentity <localidentity2@gmail.com>
We were not taking into account the ailment avoidance from Ancestral Vision when determining if the Skitterbots effects should be applied to the player
Co-authored-by: LocalIdentity <localidentity2@gmail.com>
Adds support for all the minions granted by the ascendancy and also the 5 legacy skills granted by Tattoo's that I believe still exist in standard and may come back in the future at some point
Adds support for tracking the number of each Tattoo type to properly scale the buff effects too
Co-authored-by: LocalIdentity <localidentity2@gmail.com>
* Add support for average Lightning Tendrils DPS
Lightning Tendrils has a mechanic where it critically strikes every 3rd / 5th use but we were did not have a averaged calc for this
There is now a new skill part for the averaged DPS
I also applied DPS multipliers to the other 2 skill parts to show accurate DPS for just those parts when they hit as people may use the stronger pulse one to see DPS for Ignite
* Fix formula for average damage on Eccentricity
Averaging the crit chance and damage was not correct to get an accurate DPS value
---------
Co-authored-by: LocalIdentity <localidentity2@gmail.com>
Poisonous Concoction of Bouncing gains DPS from extra chains as the projectiles keep chaining to the same target
Co-authored-by: LocalIdentity <localidentity2@gmail.com>
* Add text after red mods to show that PoB does not currently handle them
PoB does not calculate mods that have red text on them but many users do not know this. Adding this reminder text after all the lines should make it more obvious
* Fix first load crash
* Fix check for empty line
* Typo
* FIX: fixes toggle requiring reload, minor optimalizations (#8928)
The TooltipClass:CheckForUpdate had to be updated so that
the tooltip updates correctly after changing the option.
I kinda obsessed over optimizing this. Removed the secondary
loop as the first one without the break statement will
update all the records. Though this has the cost of doing
more comparisons which could end up being more expensive if
strings end up being compared often. Looking through current
calls to this function it seems like it's all just addresses
and booleans.
---------
Co-authored-by: LocalIdentity <localidentity2@gmail.com>
Co-authored-by: Paliak <91493239+Paliak@users.noreply.github.com>
Hovering over an unallocated node in radius of Impossible Escape or
Intuitive Leap currently shows difference for taking a path from the
class start which it doesn't require and won't take on click.
This is confusing and not helpful because highlight for the pathing
is being suppressed already.
The keystone effects were applied in mod parser so if you had more than 1 copy of the mod in your build it would apply multiple times
Co-authored-by: LocalIdentity <localidentity2@gmail.com>
Enemy ailment immunity or chance to avoid ailments was not factored into the sources of guaranteed ailments or chance ailments like ignite
Using Purity of Elements with the Runegraft now correctly disables all sources of elemental ailments on enemies
Co-authored-by: LocalIdentity <localidentity2@gmail.com>
output.Duration is already modified by debuffDurationMult in the duration block above so it doesn't need to be applied here again
Co-authored-by: LocalIdentity <localidentity2@gmail.com>
* Add support for Defiance of Destiny Gain before enemy hit
* update config and comment wording
* update config option label
* fix merge issues
* fix more merge issues
---------
Co-authored-by: LocalIdentity <31035929+LocalIdentity@users.noreply.github.com>
* FIX(mirage): fix cap GC cooldown by mirage+attack delay
The Howlcrack unique item sets the cooldown of warcies to 0 causing the
GC dps to explode into infinity due to division by zero. This was caused
by the assumption that the mirages instantly spawn and attack. This
commit makes it so that the cooldown is at least the amount of time it
takes to spawm them all and attack or channel for 1s.
* FIX: cleanup handling of limitedSkills table
* FIX: cleanup target skill selection for SacredWisps and MirageArcher
* FIX: mark the cache generated by mirage calcs as CALCULATOR
The env used by mirage skills is often modified causing the resulting
env to differ. This has not caused issues to there not beign any caching
use after calcMirages but this could come up in the future. Marking the
env as CALCULATOR still allows for the env to be cached in case it's
really needed but it will prevent it being used in most cases.
* FIX: remove vestigial CACHE table
This was once used as the main table holding the cached output of
skills. It has been entirely replaced by the remaining tables.
* MISC: typo
* FIX: typo
* Fix Volatility applying to Minion Spell skills
Volatility was not being restricted to minion spells after we recently changed how we were applying support gems to minions
---------
Co-authored-by: Paliak <91493239+Paliak@users.noreply.github.com>
Co-authored-by: LocalIdentity <localidentity2@gmail.com>
* Added beastcrafts to item crafter
* Fixed a formatting issue
* Formatting
* Fix end block
* Remove Flask mods as they are the exact same as t3 regular mods
* Move
---------
Co-authored-by: LocalIdentity <localidentity2@gmail.com>
* FIX(mirage): fix cap GC cooldown by mirage+attack delay
The Howlcrack unique item sets the cooldown of warcies to 0 causing the
GC dps to explode into infinity due to division by zero. This was caused
by the assumption that the mirages instantly spawn and attack. This
commit makes it so that the cooldown is at least the amount of time it
takes to spawm them all and attack or channel for 1s.
* FIX: cleanup handling of limitedSkills table
* FIX: cleanup target skill selection for SacredWisps and MirageArcher
* FIX: mark the cache generated by mirage calcs as CALCULATOR
The env used by mirage skills is often modified causing the resulting
env to differ. This has not caused issues to there not beign any caching
use after calcMirages but this could come up in the future. Marking the
env as CALCULATOR still allows for the env to be cached in case it's
really needed but it will prevent it being used in most cases.
* FIX: remove vestigial CACHE table
This was once used as the main table holding the cached output of
skills. It has been entirely replaced by the remaining tables.
* MISC: typo
* FIX: typo
Kalandra's Touch now copies the influence from the other ring
Also realised that this has always been slightly broken as it was not using the correct names for the conqueror mods
Co-authored-by: LocalIdentity <localidentity2@gmail.com>
We didn't update Cinderswallow when it was changed in 3/26 to add some new mods and remove old ones
Co-authored-by: LocalIdentity <localidentity2@gmail.com>
The Starting nodes from ascendant are within the radius of the jewels and were not checking for the ClassStart attribute to be false
Co-authored-by: LocalIdentity <localidentity2@gmail.com>
Ancestral Bond does not apply to mirages
Not 100% if this is the best place to put the condition
Co-authored-by: LocalIdentity <localidentity2@gmail.com>
In 3.26 it was changed so that Devouring Diadem uses prefix veiled mods instead of suffix ones
Also added support for the crit damage mod
Co-authored-by: LocalIdentity <localidentity2@gmail.com>
We never updated the unique talismans with their catalyst tags after tainted catalysts were added to the game
Co-authored-by: LocalIdentity <localidentity2@gmail.com>
Linking Enfeeble to High Impact Mine caused sources of Increased Mine Aura effect to scale the curse effect on Enfeeble
Co-authored-by: LocalIdentity <localidentity2@gmail.com>
