PathOfBuilding icon indicating copy to clipboard operation
PathOfBuilding copied to clipboard

Kitava's Thirst trigger

Open creativity-x opened this issue 2 years ago • 6 comments

Check for duplicates

  • [X] I've checked for duplicate issues by using the search function of the issue tracker

Is your feature request related to a problem?

I am trying to trigger Kitava's Thirst using the surging thoughts harbinger sword, spending 100 mana by triggering the weapon spell socketed by Archmage support. The helmet proc trigger is the top line, but it does not trigger off the correct mechanic, and is not able to be modified as far as I can see, making the DPS calculation inaccurate.

Describe the solution you'd like

Add a manual option for Kitava's Thirst Trigger rate

Describe alternatives you've considered

No response

Additional context

No response

creativity-x avatar Aug 29 '23 02:08 creativity-x

This sound more like a bug report, as it should work automatically AFAIK. Could you provide a build code showing the problem please?

Nightblade avatar Aug 29 '23 02:08 Nightblade

Yes sorry it is a bug report, I don't know what I'm doing on github, so I don't know if I can change that

Here is the build.... look at the left panel top line in blue text: "Kitava's Thirst Trigger" https://pobb.in/SM2wy-QJURUh

I think in this interaction, the trigger rate should not be incorrectly tied to "cast speed" and rather be off Storm Cascade, triggered off any attack skill (which has 0.25s base internal cooldown; Kitava's Thirst has 0.1s ICD). I also think due to this, Choir of the Storms dps calculation in this situation is not correct either.

Thanks for your help

creativity-x avatar Aug 29 '23 12:08 creativity-x

Two issues:

  • You don't have enough unreserved mana for archmage to increase the cost of Storm Cascade to 100.
  • Nested triggers often have cyclic dependencies and those have a tendency to cause infinite recursion crashes. For most triggers it's currently not possible in POB to trigger a trigger with a triggered skill for this reason.

I have thought of implementing a sort of maximum recusion depth for triggers to allow for this kind of interactions but i'm not sure if or when i'll work on it.

Adding the ability to select a source skill from a filtered list would also be a nice feature but again. Not sure when or if.

Paliak avatar Aug 29 '23 14:08 Paliak

Hi: 1.) Even with enough unreserved mana the issue with the interaction still exists. After getting rid of divine blessing, POB simply reports "No Kitava's Thirst Trigger". 2.) Completely makes sense. I assumed the lag was due to spellblade support. Maybe its incalculable in all honesty, its so many variables and triggers happening at once. I would say there's no need to work on anything. I will continue to just wing it and go by feel lol

Thanks again

creativity-x avatar Aug 29 '23 15:08 creativity-x

1.) Even with enough unreserved mana the issue with the interaction still exists. After getting rid of divine blessing, POB simply reports "No Kitava's Thirst Trigger".

You need to remove some of your auras to make spell cascade cost enough mana but even then it won't be picked up due to issue 2) .

The issue with 2) is not really about the implementation but more about the performance as an initial sketch made adding gems to groups take good 40 sec. Could be improved but yeah.

Paliak avatar Aug 29 '23 15:08 Paliak

@creativity-x Is this OK to close now?

Nightblade avatar Aug 29 '24 05:08 Nightblade