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.

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

Fix supplementalDamage calc #351

Merged

Conversation

kagekiyo7
Copy link
Collaborator

When other than OugiDamage, supplementalDamage is not affected by damageUP.

https://twitter.com/zekasyuz/status/1075358728029761537

エッケザックス added 2 commits September 11, 2019 03:21
When other than OugiDamage, supplementalDamage is not affected by damageUP.
@kei-gbf
Copy link
Collaborator

kei-gbf commented Sep 28, 2019

  • appply damageUp only ougiDamage?
    • chainBurst was not mentioned in the link (require veritication)
  • "修羅の誓約" 'contentious' is for TA only not have ougiDamage ... ok

@kei-gbf kei-gbf added the bug label Sep 28, 2019
@kagekiyo7
Copy link
Collaborator Author

chainBurst was not mentioned in the link (require veritication)

I don't know. however, he said that's ougi only.

-奥義ダメージの場合、与ダメージ上昇に属性特攻(与ダメUP)が有効
https://twitter.com/zekasyuz/status/1076304834838507521

@kei-gbf
Copy link
Collaborator

kei-gbf commented Sep 28, 2019

so, then it's missing info about chain burst.

This PR patch changes supplemental damage formula

  • no damageUp for normal attack supplemental ... ok
  • keep damageUp for ougi supplemental ... ok
  • no damageUp for chain burst supplemental ... not sure
    the case was not mentioned explicitly in the verification source.

@kagekiyo7
Copy link
Collaborator Author

I create issues. #360

@kei-gbf
Copy link
Collaborator

kei-gbf commented Sep 29, 2019

This comment is re-post
Deleted the same comment on #360, the issue seem list for require verification
here is the topic specific thread.

no damageUp for chain burst supplemental damage ?

チェインバーストの与ダメージ上昇に天司等の与ダメージUPが有効かどうかの検証

  • 虚空弓 C.B でのクリティカルは発生しないはず?なので、条件外
    akasha spear .. C.B does not have critical, out of the target
  • 虚空斧 TA時の為、条件外
    akasha axe .. C.B does not have TA, out of the target

ランサー(C.B関連のLB振っていない) + 闇天司斧5凸メイン + 虚空槍
Rキャラと2人で試してますが計算式がよくわからずでした。
近似値は得られたものの完全一致は得られず。何か、前提条件に見落としはあるかもしれない。

I had tried 2 chain Lancer (no LB) + dark seraphic axe 5stars main + akasha spear with R chara

  • Sample data Djeeta : 63880, R Chara: 6655, C.B: 56451
    • (63880 + 6655) / 4 + 30000 = 47633 <- 与ダメアップが乗らない場合 no damageUp
    • (63880 + 6655) / 4 + 30000 x 1.23 = 54533 <- 与ダメアップが乗る場合 with damageUp
    • ((63880 + 6655) / 4 + 30000)x 1.23 = 58589 <- I don't know, just test this formula

もうひとつ気になったのは、虚空武器+チェインバーストの場合は
誰の得意武器が適応されるのか・・・主人公?

ランサー(槍得意) 通常攻撃 + Rキャラ(虚空槍対象外)2人で2chain したところ
主人公通常攻撃: 35351, Rキャラ奥義2人 6447, 10920 C.B: 36513
※ 30000 x 23% = 36900 なので、ここだけ見ると与ダメUP込みの値を下回っている

@kei-gbf
Copy link
Collaborator

kei-gbf commented Sep 29, 2019

有利属性(x1.5)だったのかな。

Sample data Djeeta : 63880, R Chara: 6655, C.B: 56451

  • (63880 + 6655) x 1.5 / 4 + 30000 = 56450.625 小数点以下を切上で一致

@kei-gbf
Copy link
Collaborator

kei-gbf commented Sep 29, 2019

主人公通常攻撃: 35351, Rキャラ奥義2人 6447, 10920 C.B: 36513

  • (6447 + 10920) x 1.5 / 4 + 30000 = 36512.625 切り上げで一致

thus no damageUp to chainBurst supplemental damage

damageUp は有利でないといけないのを失念してました。
チェインバーストの与ダメ上昇に与ダメアップ(damageUp) は載らないのを確認。

虚空剣でも同様。他の与ダメ上昇バフは試してません。

もうひとつ気になったのは、虚空武器+チェインバーストの場合は
誰の得意武器が適応されるのか・・・主人公?

  • サブ虚空槍
    • ファイター(通常攻撃) 刀槍得意の奥義で2チェインでは、
      チェインバーストの与ダメ上昇30000は得られず。

new issue?
for chainBurst, Djeeta's supplemental damage was used

Copy link
Collaborator

@kei-gbf kei-gbf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Confirmed: no damageUp affected to chainBurst supplemental damage.

@kagekiyo7
Copy link
Collaborator Author

検証ありがとうございました!


ファイター(通常攻撃) 刀槍得意の奥義で2チェインでは、
チェインバーストの与ダメ上昇30000は得られず。

for chainBurst, Djeeta's supplemental damage was used

OKです。その辺りも修正PR作ってみます。

@kagekiyo7 kagekiyo7 merged commit 37106d3 into MotocalDevelopers:master Sep 30, 2019
@kagekiyo7 kagekiyo7 deleted the Fix-supplementalDamage-calc branch September 30, 2019 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants