Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Core/Threat] SpellThreat-Calculation revamped #2881

Closed
wants to merge 1 commit into from
Closed

[Core/Threat] SpellThreat-Calculation revamped #2881

wants to merge 1 commit into from

Conversation

Sarjuuk
Copy link
Contributor

@Sarjuuk Sarjuuk commented Sep 3, 2011

short:

  • Threat is now distributed blizzlike (almost)

long:

  • Percent- and AP-Modifieres added
  • Threat is now properly distributed through multiple enemies
  • Most spells will cause initial threat in an amount equal to their SpellLevel
  • Energize-Effects will now be treated like healing (threat-wise)
  • Tanks got their missing +43% passive threat tacked on
  • Paladins are still missing 'Salvation' and will not only cause half threat with healing (essentially 25% of the amount healed)

@Sarjuuk
[Core/Threat] SpellThreat-Calculation revamped
f728c91 
short:
  - Threat is now distributed blizzlike (almost)

long:
  - Percent- and AP-Modifieres got add
  - Threat is now propperly distributed through multiple enemies
  - Most spells will cause initial threat in an amount equal to their SpellLevel
  - Energize-Effects will now be treated like healing (threat-wise)
  - Tanks got their missing +43% passive threat tacked on
  - Paladins are still missing 'Salvation' and will not only cause half threat with healing (essentially 25% of the amount healed)
@SignFinder
Copy link
Contributor

mmmm good commit

@gyullo
Copy link
Contributor

gyullo commented Sep 3, 2011

Im using it and work excelent
+1 to repo
+2 for a good work

@Darkrabbit
Copy link

+100 to repo XD

Work Fine Thanks Man!!

@QAston QAston closed this Sep 3, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@Sarjuuk @SignFinder @gyullo @Darkrabbit @QAston