Giter Site home page Giter Site logo

🐛 [Bug] Missed Crit Suppression about core HOT 1 OPEN

sh3ppard avatar sh3ppard commented on June 28, 2024
🐛 [Bug] Missed Crit Suppression

from core.

Comments (1)

sh3ppard avatar sh3ppard commented on June 28, 2024

I was wrong. It implemented, I found it in code:

core/src/game/Objects/Unit.cpp

Lines 2688 to 2693 in 5d27d09

// Apply crit chance from defence skill
int32 skillDiff = int32(GetWeaponSkillValue(attackType, pVictim)) - int32(pVictim->GetDefenseSkillValue(this));
int32 minSkillValue = GetSkillMaxForLevel(pVictim) < GetWeaponSkillValue(attackType, pVictim) ? GetSkillMaxForLevel(pVictim) : GetWeaponSkillValue(attackType, pVictim);
int32 cappedSkillDiff = minSkillValue - pVictim->GetDefenseSkillValue(this);
crit += (pVictim->IsPlayer() || skillDiff > 0) ? skillDiff * 0.04f : cappedSkillDiff * 0.2f;

Also need to research about the crit suppression modifier which placed on total Critical Strike:

Modifiers to Critical Strike chance are reduced against enemies 3 levels higher than the player-character. There is a single modifier placed on total Critical Strike gained from auras that increase Crit chance.

Unfortunately Blizzard doesn't share that value.
image

Source

But guys from fightclub did some great work:

In addition, when fighting +3 level mobs there's a flat modifier placed on your total crit chance gained from auras. Auras in this context are talents such as Cruelty or Axe Specialization, gear that directly gives crit % through Equip: effects, buffs that increase you crit chance like Songflower Serenade or Leader of the Pack, and consumables such as Elixir of the Mongoose. It does not include crit gained indirectly through Agility, neither base Agility or Agility from gear.

While Blizzard only revealed the nature of this modifier but not its value, we've done extensive research with upwards of 60K hits and concluded that the modifier is most likely a flat 1.8% reduction to your crit chance gained from auras. If you don't have any crit from auras you are not affected by this, but still affected by the 3% suppression due to skill/defense difference mentioned above. Realistically, you will most likely have at least 2% crit from auras through one of the sources mentioned above even while leveling past level 10 (think 2/5 Cruelty) and certainly at level 60, putting your total suppression against +3 level mobs at 4.8%.

Source

This modifier should be ~1.8%.

from core.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.