View Issue Details

IDProjectCategoryView StatusLast Update
0001440Legion CoreClasses - Demon Hunter - Охотник на демоновpublic2019-03-27 12:06
ReporterenfaimAssigned ToGhost 
PriorityhighSeveritymajorReproducibilityalways
Status resolvedResolutionfixed 
Summary0001440: (Охотник на демонов) Критический хаос
Descriptionhttps://ru.wowhead.com/spell=201455
Бонус от перка у нас аддитивно влияет на мод крит. урона: 2 + (1 + 0.06 * ArtifactTraitRank(CriticalChaos)), а должен влиять мультипликативно: 2 * (1 + 0.06 * ArtifactTraitRank(CriticalChaos)).
У нас:
http://egammi.com/image/SNyt при 6 рангах (36% увеличение крит. урона) соотношение усредненного крита к хиту у Аннигиляции 924004/398529 = 2.32
http://egammi.com/image/SNyj у Удара хаоса примерно та же картина 715683/313246 = 2.29
Оф. сервер:
https://www.warcraftlogs.com/reports/X26Vgj7kHnJ1zByQ#fight=18&type=damage-done&source=13&targetbuffs=206491
У ДХ 42% увеличение крит. урона УХ/Аннигиляции (7 рангов перка)
Считаем соотношение урона усредненного крита к хиту у:
Аннигиляция правая рука 2147925/767371 = 2.80
Аннигиляция левая рука 1711951/598667 = 2.86
УХ правая рука 1621589/548154 = 2.96
УХ левая рука 1289669/437312 = 2.95
Выборка по количеству приемлемая, так что картину с перком обрисовывает наглядно. Т.е. модификатор крит. урона для УХ/Аннигиляции в случае с 7-ю рангами перка должен составлять 2*(1+0.06*7) = 2.84, с 6-ю рангами 2.72 и т.д.

Activities

enfaim

2019-02-28 22:18

reporter   ~0001067

Поправка: ... у нас аддитивно 2+(0.06*ArtifactTraitRank(CriticalChaos)),... а должно быть как описано выше...

enfaim

2019-02-28 23:44

reporter   ~0001070

Сюда же https://ru.wowhead.com/spell=243188, соотношение между критом/хитом у нас сейчас 2.2, должно быть 2.4 т.к. мультипликатив тут у перка также должен быть...был на форуме уже раньше репорт на это с развёрнутыми пруфами. Сюда же и https://ru.wowhead.com/spell=200373, соотношение щас 2.25, должно быть 2.5.

user34

2019-03-01 03:40

  ~0001080

enfaim видимо воины тоже просели с их https://ru.wowhead.com/spell=200860/ при 7/7 трейте, должно быть 2*1.35=2.7 мод крита.

Chio

2019-03-01 07:28

reporter   ~0001091

И афлики тоже...видимо, недавняя правка по крит модам была ошибочной

Ghost

2019-03-01 12:18

developer   ~0001100

отвечу за все подобные темы

у элема 60188 как добавляет крит к 200% делая 250% ? аддитив... аддитив
210575 у ферала
https://www.warcraftlogs.com/reports/3bqmBYrvTadC7FAc#fight=7&type=damage-done&source=12&view=events&ability=22568
5 рангов ауры но крит множитель равен 2.35, а не 2.7, а эффект такой же, что у "критического хаоса" странно да? по сути бп должно было быть 1.35 и стать 2.7, но оно добавилось к 200% и стало 235%, значит так и должно быть
у перка дх он будет 242%, не 284% и это логично


теперь другое, вот список аур, у которых бп близзы задавали специально вдвое больше, вот тут нужна инфа, действительно ли это правдоподобно, что здесь сам крит урон растет, а не критмножитель увеличивается аддитивно
https://ru.wowhead.com/spell=189144
https://ru.wowhead.com/spell=190449
https://ru.wowhead.com/spell=191572
https://ru.wowhead.com/spell=195322
https://ru.wowhead.com/spell=195323
https://ru.wowhead.com/spell=196217
https://ru.wowhead.com/spell=199158
https://ru.wowhead.com/spell=200315
https://ru.wowhead.com/spell=238053
https://ru.wowhead.com/spell=238067
https://ru.wowhead.com/spell=238068
https://ru.wowhead.com/spell=254332
https://ru.wowhead.com/spell=200376
можете по каждой ауре четкую инфу найти? делалось не мной) не советую искать логи на аурах , где бонус действует на спеллы с увс от оружия, ибо разброс дает путаницу в разности крит эффект урона...

Ghost

2019-03-01 12:26

developer   ~0001102

https://ru.wowhead.com/spell=200860 https://ru.wowhead.com/spell=209417 также не попадали под это, здесь четко понятно, что урон от крита растет
да и эффект здесь другой

Ghost

2019-03-01 12:31

developer   ~0001103

это ими делалось потому, что эффект то работает с критмножителями, а по описанию они крит урон/лечение увеличивают и чтобы не менять каждому эффект, увеличили им бп в 2 раза и получается при аддитивном действии допустим ауры 200376 крит урон/хил равен 250% от урона/хила хита

Chio

2019-03-01 13:48

reporter   ~0001104

Для начала - немного лирики. Во времена лич кинга приходилось лезть на ВХ искать английское описание, по той причине, что русский перевод пестрил косяками и нестыковками. Для выяснения что да как и почему, пришлось малость поинтересоваться английским, и вот что из этого получилось:
Актуальная инфа: есть два типа описания по криту - 1) Увеличивает бонусный урон от крита и 2) Увеличивает критический урон / урон при критическом попадании.
(тут в английских описаниях все четко)
В первом случае описание необходимо воспринимать следующим образом: есть базовый урон - 100%. Есть "бонусный урон при критическом попадании". И тут сразу очевидно, что эти два понятия аддитивны, т.е. к базовому урону прибавляется бонусный урон от крита, и, соответственно, если есть аура, увеличивающая бонусный урон от крита - то на выходе получаем простую сумму - аддитивное отношение: базовый урон + (бонусный урон от крита + аура).
Во втором случае эффект действует не на бонусный урон от крита, а на все значение крита, и тогда эффект является мультипликативным: (базовый урон + бонусный урон от крита) = урон от крита. Конечный урон = урон от крита * эффект (точнее, (1 + эффект / 100%).
Не уверен, что на легионе все именно так и осталось в плане описаний, но все же.
Пока что почекал вовлогс - следующие эффекты мультипликативны - Ад (афли), Мокрое место (фдк), Холодное сердце (фрост маг), Критический хаос (дх). Остальные не смотрел.

Ghost

2019-03-01 14:50

developer   ~0001108

Chio не надо разжевывать лирику, я задал конкретный вопрос, остального касаться не будет)

enfaim

2019-03-01 15:01

reporter   ~0001109

Ghost Ферал это ферал, репорт не по нему сделан.
>у перка дх он будет 242%, не 284% и это логично
Вот только логи обратное доказывают. На офе при 6/6 трейтах крит множитель 2.72, у нас же 2.36, разница очевидна.
Разброс урона у оружия оказывает влияние, но для того я и указываю усреднение + привожу только те где меньше рандомности (без проковых тринь, события отфильтровываю по 206491 ауре), да и огромное кол-во логов показывают одну и туже картину, не нашел еще ни одного где бы критмножитель был близок к тому, о котором вы говорите (при условии достаточно большой выборки).
Еще рандом с Вариматраса:
https://www.warcraftlogs.com/reports/DpBbkRdV17wjctvh#fight=41&type=damage-done&source=27&targetbuffs=206491&ability=-201427 с аурой 206491
https://www.warcraftlogs.com/reports/DpBbkRdV17wjctvh#fight=41&type=damage-done&source=27&targetbuffs=-206491&ability=-201427 без ауры 206491
Крит множитель у правой/левой руки находится в пределах 2.68-2.77 (у дх 6/6 трейтов), что оч. близко к 2.72 нежели к 2.36.
По поводу списка аур, ранее на форум я кидал уже логи по https://ru.wowhead.com/spell=200376, когда стоял вопрос о https://ru.wowhead.com/spell=243188.
https://www.warcraftlogs.com/reports/qrV61cb2n7HpkJTG#fight=2&type=healing&source=10&ability=183811&view=events&sourcebuffs=31842.200376
00:03:04.662 Falsudin Judgment of Light Matohd +*123539*
00:03:04.662 Falsudin Judgment of Light Matohd +49416
По 243188 аналогично, критмножитель 2.4 должен быть, а не 2.2 как сейчас.

Ghost

2019-03-01 15:32

developer   ~0001112

И что, что не написан? я же говорю, что это общий эффект, у них такое бп задано в клиенте. В противном случае это катастрофически сказывается на уроне и байта и ух, они уже выходят из симкрафтовых значений, в пвп вообще это сразу будет заметно, т.к там крит множитель еще ниже. Учитывая то, что в логах ух критует парой ударов, то мы вообще ему должны переставать верить.
И будут у нас байты лететь по 2млн и выше, а ух критовать на 1.5+ (c двух ударов) . Это нормально будет?
И нельзя на усредненных значениях производить расчеты, потому что нельзя начисто вывести урон базовый, вы не знаете какой там урон пролетел, максимальный или минимальный диапазона, вдруг его там не было вообще? я поэтому говорю, не стоит искать пруфов на увс от оружия... достаточно уже напримере лога байта (у которого базовый урон без диапазонов и он не увс от оружия) я показал это, могу на примере лога т21 шп это показать, там крит урон для 8092 тоже аддитивно увеличивается и равен 2.5, а не 3.0
И не надо думать, что там для разных перков по своему считается, я логику раскрыл аур такой логики, у них одна общая логика у всех абсолютно

enfaim

2019-03-01 18:28

reporter   ~0001118

Что странного в логике одновременных критов у УХ и почему мы им не должны верить? Близы, имхо, сделали это специально, т.к. иначе искажается правильная работа УХ с точки зрения восполнения гнева. Представим, что у персонажа 50% крита, то по офовской реализации всё просто - вероятность того, что за применение УХ игрок восполнит 20 ед. гнева, равна 50%, т.о. вероятность крита = вероятности восполнить гнев. Если бы они сделали раздельный шанс крита для правой/левой руки как у нас, то вероятность восполнить гнев согласно теории вероятности возрастёт и составила бы уже 75%, т.к. у нас может уже не только обе руки критануть одновременно, а например комбинация: либо правая/либо левая рука, т.е. появляется два дополнительных возможных события, при которых восполняется гнев.

По поводу перка. Вам может быть и достаточно примера байта в качестве пруфа для всех остальных перков данного типа, а мне нет (да и не мне одному), т.к. логи говорят о другом. Либо приводите логи, в которых на достаточно большой выборке было бы видно, что крит. множитель был аддитивен. Не надо скидывать всю вину на разбег урона оружия, на достаточно больших выборках это усредняется и чем больше хитов тем точнее вырисовывается крит. множитель, + в логах можно также считать соотношения и по абсолютно максимальным/минимальным значениям урона.

Хорошо, в последний раз приведу наглядное соотношение в попытках указать на баг, Гост ознакомься плиз полностью:
У нас https://www.warcraftlogs.com/reports/CBWRqt1kvT7yKpba/#fight=last&type=damage-done&source=1&sourcebuffs=242584 (6 трейтов)
проковых тринь нет, отфильтрованы события по конкордансу, т.к. под ней было больше всего хитов нанесено
Аннигиляция пр. рука, min: 957636/408247 = 2.35
Аннигиляция пр. рука, max: 1091049/462830 = 2.36
Аннигиляция пр. рука average: 1033227/434381 = 2.38
для левой руки и УХ значения крит. множителя примерно такие же, т.о. с учетом разбега урона оружия и проков конкорданса показатель крит. множителя составил от 2.31 до 2.39, как видим разлёт в пределах 0.1, а должно быть соотношение 2.72.
Оф. сервер
https://www.warcraftlogs.com/reports/6HpYkn8z49GXb2dW#fight=14&type=damage-done&source=25&targetbuffs=206491&sourcebuffs=-229206.-242584
"Серые логи" дхшки с 7/7 перком, события отфильтрованы без конкорданса, без зельки, но с заклятым врагом(206491), т.к. под ним было больше всего хитов
Аннигиляция правая рука, min: 1805864/639835 = 2.82
Аннигиляция правая рука, max: 2157127/775079 = 2.78
Аннигиляция правая рука, average: 1950111/712109 = 2.74
Аннигиляция левая рука, min: 1433359/519794 = 2.76
Аннигиляция левая рука, max: 1710725/610569 = 2.80
Аннигиляция левая рука, average: 1604026/561850 = 2.85
УХ маленькая выборка с заклятым врагом, поэтому посмотрим без него
https://www.warcraftlogs.com/reports/6HpYkn8z49GXb2dW#fight=14&type=damage-done&source=25&targetbuffs=-206491&sourcebuffs=-242584.-229206
УХ правая рука, min: 1119667/399434 = 2.80
УХ правая рука, max: 1275445/452613 = 2.82
УХ правая рука, average: 1198890/426790 = 2.81
УХ левая рука, min: 903558/314585 = 2.87
УХ левая рука, max: 1014617/355952 = 2.85
УХ левая рука, average: 956272/334273 = 2.86
Т.о. крит. множитель при 7/7 рангов перка составил от 2.74 до 2.87 с учетом разбега урона оружия. Ну никак не может быть тут 2.42 множителя, от слова совсем, даже если прикидывать по самому грубому. Будем и дальше бодаться или вы примете баг? Потому как в зависимости от количества рангов у перка мы теряем от 10 до 17% урона у УХ/Аннигиляции.

Ghost

2019-03-01 19:03

developer   ~0001121

нет, не приму

Ghost

2019-03-01 19:21

developer   ~0001122

Если я нахожу логи аур,где на 100% выводится крит множитель и его алгоритм вывода сходится, значит логика всего блока ауры и спеллов, которые подчиняются его алгоритму и она совпадает с логикой реализации блока, а в данном участке кода нет хаков, то все спеллы из этого блока работают по одному принципу, а значит если корректировать одно - сломать другое, сломать святое - взятие бп из клиента. Да и пачкать ядро никто не будет
Мне со спокойной душой стоит убрать кастомщину, сделанную по списку аур, которые я линканул.

Chio

2019-03-02 07:53

reporter   ~0001156

Как пруф по афликам https://www.warcraftlogs.com/reports/7dfMjFzKW1YXkPGV#fight=1&type=damage-done&source=18&view=events&ability=198590&sourcebuffs=216708.256832.242586 примите? Взял для примера самую простую способность, которую затрагивает минимум проков, эффектов сэта и прочая.

user34

2019-03-02 08:05

  ~0001158

Chio а вот тебе другой лог
https://www.warcraftlogs.com/reports/WjQMJ9BVTgG1ZLfx#fight=31&type=damage-done&source=203&ability=198590&view=events&sourcebuffs=-216708
у человека 7/7 трейтов на АД
и соотношение крита к не криту 2.46
а такое может быть только при расчете этого модификатора как 2*1.23=

user34

2019-03-02 08:08

  ~0001160

и да, если рассматривать что 100% урона от крита это 200% от обычного, то бонусные 46% урона от обычного и будут 23% урона от крита.

user34

2019-03-02 08:17

  ~0001161

и снова тот же варлок с 7/7 ад
https://www.warcraftlogs.com/reports/WjQMJ9BVTgG1ZLfx#fight=31&type=damage-done&source=203&ability=198590&view=events
соотношение крита к не криту 2.92, а это возможно только засчет трейта артефакта урон считался вот так
2*(1+(0.23*2)=2.92
где 2* это стандартный урон крита
0.23 это 7/7 ад.
*2 влияние прожатого артефакта
1+ это стандартный модификатор крита.

Ghost

2019-03-02 13:23

developer   ~0001165

А есть логи шаблоном вообще? Как там ведут себя 24% кх

user34

2019-03-02 15:20

  ~0001167

Ghost перефразировать можешь? потому что я не совсем тебя понял, что ты просишь.. но поискать могу.

Ghost

2019-03-02 15:26

developer   ~0001168

на пвп манекене 4 ранга этого перка активны и диапазон увс от оружия 0-1 разница, там четко можно вывести этот множитель, раз от нас не отстанут, хотят бодаться, придется хакать для этой ауры в обратную сторону
ну пусть бодание будет на этом манекене, сведем разность диапазона до 1 единицы)

enfaim

2019-03-02 18:47

reporter   ~0001174

https://youtu.be/SPe8u6L8S48?t=509
крит правой рукой 621467
хит правой рукой следующий каст 250591
621467/250591 = 2,4800052675475176682322988455292

Issue History

Date Modified Username Field Change
2019-02-28 22:05 enfaim New Issue
2019-02-28 22:05 enfaim Status new => assigned
2019-02-28 22:05 enfaim Assigned To => MrSmoorf
2019-02-28 22:18 enfaim Note Added: 0001067
2019-02-28 23:44 enfaim Note Added: 0001070
2019-03-01 03:40 user34 Note Added: 0001080
2019-03-01 07:28 Chio Note Added: 0001091
2019-03-01 12:18 Ghost Note Added: 0001100
2019-03-01 12:19 Ghost Tag Attached: крит множители и урон крита
2019-03-01 12:26 Ghost Note Added: 0001102
2019-03-01 12:31 Ghost Note Added: 0001103
2019-03-01 12:58 MrSmoorf Status assigned => acknowledged
2019-03-01 13:48 Chio Note Added: 0001104
2019-03-01 14:50 Ghost Note Added: 0001108
2019-03-01 15:01 enfaim Note Added: 0001109
2019-03-01 15:32 Ghost Note Added: 0001112
2019-03-01 18:28 enfaim Note Added: 0001118
2019-03-01 19:03 Ghost Note Added: 0001121
2019-03-01 19:21 Ghost Note Added: 0001122
2019-03-02 07:53 Chio Note Added: 0001156
2019-03-02 08:05 user34 Note Added: 0001158
2019-03-02 08:08 user34 Note Added: 0001160
2019-03-02 08:17 user34 Note Added: 0001161
2019-03-02 13:23 Ghost Note Added: 0001165
2019-03-02 15:20 user34 Note Added: 0001167
2019-03-02 15:26 Ghost Note Added: 0001168
2019-03-02 18:47 enfaim Note Added: 0001174
2019-03-03 00:31 Ghost Assigned To MrSmoorf => Ghost
2019-03-03 00:31 Ghost Status acknowledged => resolved
2019-03-03 00:31 Ghost Resolution open => fixed
2019-03-27 11:21 Eva Category Classes / Классы => Classes - Death Knight - Рыцарь смерти
2019-03-27 12:06 ThisMushroom Category Classes - Death Knight - Рыцарь смерти => Classes - Demon Hunter - Охотник на демонов