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

Add default values and Adapt kouzin job bonus #192

Merged

Conversation

kagekiyo7
Copy link
Collaborator

@kagekiyo7 kagekiyo7 commented May 13, 2019

・Rank100辺りを想定にデフォルト値を追加(データビューアを見ていると特に召喚石合計攻撃力など入力しないない方がちょくちょくいるので)
・攻刃ジョブボーナスが動作していなかったので適応

@kei-gbf
Copy link
Collaborator

kei-gbf commented May 14, 2019

初期値は、用途やランク帯・装備でいろんな立場からの見方がありそうですが

防御デバフの初期値に関しては、
初期値が知らない間に変わってると影響出そうな人がいるのと、
(これは、更新履歴・お知らせは読まれない人想定ですが。。)

装備が減衰に届かない状況では、計算結果の武器優先度に影響が出にくい項
等を顧みて、検討しても良いでしょうか。

現在の初期値 提案 影響
Rank 1 100
防御デバフ 0 50 x2
召喚石Attack 0 9500
召喚石HP 0 4000
フレ石(属性) 80 140 有利 x 1.26

今の段階では変更要求というわけではなく、
ここに関してはもう少し書きたいことあります。

EDIT: 表修正 Rank
EDIT: Attack/HP は召喚石


自分で使いたい場合は、テンプレを用意する等して、
運用でカバーできるので個人的には問題なしです。

@kagekiyo7
Copy link
Collaborator Author

OKです。よろしくお願いします。

@kei-gbf
Copy link
Collaborator

kei-gbf commented May 15, 2019

デバフ無し

  • 予測ダメージが 防御値10(トライアル) デバフ無し
  • 周回では、時短の為にデバフのアビポチは省くケースが多い。

デバフ 50%

  • ソロ/マルチ救援

実際の計算で多いのは50だと思いますが、初期値変更による影響を考えると

Rank等は初期値がいつの間にか変わってても影響は軽微だけど、
防御デバフ50はダメージが倍変わってくるので影響大きそうで、(上限アップ系の優先度)
変更を知らずに計算する人が出てくるのではないかという点が懸念。

Rank100想定(減衰に届きにくい)で、
デバフの有無が武器の優先度に与える影響があまりないなら、
ゲーム内で表示される予測ダメージに近い、防御値10/デバフ0 でも良いかなと思ったり

ただ、私はそこまで計算機を使い込んでない為、実際の使用感をあまり把握してませんが
毎回防御値を変更するのが煩わしいと感じてるユーザが多いなら、
変更するメリットは有ると思います。

@kei-gbf
Copy link
Collaborator

kei-gbf commented May 15, 2019

デフォルトの動作を変える時、
影響が大きそうなときはユーザへの告知があった方が良いと思うのですが

変更点を周知させる手段が…
更新履歴が目につきにくいところにあるのが問題な気がしてきた。
変更点の詳細って、更新履歴に書ける事にも限りがあるし。

ユーザ視点で、例えば
いつもとダメージが倍違うけど大丈夫なのか…となって調べても、
GitHub の コミットや PR 迄たどり着けないと詳細な情報がない状況です。


  • デバフは 0 のままにする。(トライアル 防御値10/デバフ無, 予測ダメージ 想定)

  • 50に変更する場合は、Rankや防御デバフ等の初期値を変更。と更新履歴で告知。

    ※ 防御デバフの影響が一番大きいと思う為。
    他はあまり細かく詳細に書きすぎても情報が埋もれてしまうので、適度に。

@kei-gbf
Copy link
Collaborator

kei-gbf commented May 15, 2019

毎回同じ値を入力する手間に関しては、何処を想定するかは
人により変わってくる部分なので。他の案

  • よく使われる編成データを用意して、テンプレートとして活用
    • 用途に応じて初期値で開けるようにメニューを作る
  • 編成データの部分読み込み Partial Load feature #177

@kagekiyo7
Copy link
Collaborator Author

うーん、確かに色々メリット・デメリットがありそうなので一旦コミット消しましょうかね?

@kei-gbf
Copy link
Collaborator

kei-gbf commented May 15, 2019

恐らくですが、入力欄を全部埋めてないのは、
正確なダメージ量を把握するよりも、大雑把に
武器の優先順位(必要本数?)を見たいだけではないかなと思います。


フレ石の影響 80 → 140

(150 + 140) / (150 + 80) = 約1.26 (訂正 120 -> 140)

元々の初期値は属性80なので、
ここは明らかに現状と合ってない部分ですが…
課題は、変更による影響のデメリット部分をどうするか。

  • 変更の告知手段の改善
  • 優先度の低い手間のかかる要望一覧 #151 プリセット案の実装で選べるようにする
  • 他案では、テンプレート選択や、ウィザード形式
    推奨入力項目だけアンケートフォームみたいなものに入力してから初期データ生成
    各項目に目を通すことになるので、知らない間に変わっていたという状況は防げる。

初期値変更に関して、 防御デバフのみ影響が大きいのが気掛かりなので、
他の初期値の変更は、影響が少なければいいのではないかなと思います。
(ただ、影響が少なければ変更するメリットも薄い)

なるべく広く意見を募りたい部分ですね、この辺りの設定に関しては。

@kei-gbf
Copy link
Collaborator

kei-gbf commented May 15, 2019

ランクの初期値を 1 にするメリットは、ランクが入力されている場合
利用者が意図的に値を変えるわけですから、初期値から変え忘れているのを取り除けば
実際に使ってる人のランク帯がある程度把握できるかもしれない。…といった用途もあります。

デメリットで、もし統計をとったりしてると影響有り。
見た感じ、100以下のデータは見られなかったので影響は軽微ですが。

@kei-gbf
Copy link
Collaborator

kei-gbf commented May 15, 2019

Adapt kouzinBonus of job

こちらは問題なしです。今まで影響があったのはオーガのみなのかな。

初期値に関しては、取り下げ
もしくは影響が大きそうな防御デバフのみに配慮する形ならいいのではないかなと。

フレ石は、石加護は設定忘れも少なく、初期値が変更されても目につきやすいはず。

@kei-gbf
Copy link
Collaborator

kei-gbf commented May 16, 2019

初期値に関して、代案を出しました #201

@kagekiyo7
Copy link
Collaborator Author

まだそのPRの内容を把握できていないのですが、こちらのSet to defaultが要らなくなったということでしょうか?

@kagekiyo7
Copy link
Collaborator Author

kagekiyo7 commented May 16, 2019

こちらは問題なしです。今まで影響があったのはオーガのみなのかな。

そもそも守護と攻刃はジョブボーナスでは貰えず、基本的にサポアビでの効果なのであまりないですね

@kei-gbf
Copy link
Collaborator

kei-gbf commented May 16, 2019

直接的な解決ではないのですが、
URLにパラメータ指定で(サーバに設置した) JSONファイルを開けるようにしました。
現状 ?id= でサーバ上のデータを読み込めますが、こちらは JSON ファイルを対象とし

応用で、例えば、トップページを作って、ユーザが初期値を選べるようにしたり、
初期値のプリセットを呼び出すメニューを作ったり。…する為の、基礎の部分です。
(具体的なUIは考えてません)

レビューの際のデバッグでも、./json/ にテスト用のデータを蓄積すれば。
過去の編成データを開けるかどうかのテストがやりやすくなります。

@kei-gbf
Copy link
Collaborator

kei-gbf commented May 16, 2019

set to default

競合はしないのですが、初期値は、運用・ユーザに委ねる形になりますね。

個々の内容で見ると、ランクや防御デバフについては先述した通りで、
召喚石の項目のみ、現状に合わせて更新でもいいのではないかと思います。

@kagekiyo7 kagekiyo7 force-pushed the add-default-and-kouzin-jobbonus branch from ab1e7a0 to 1258424 Compare May 16, 2019 17:16
@kagekiyo7
Copy link
Collaborator Author

kagekiyo7 commented May 16, 2019

なるほどなるほど、応用次第で色々できそうな機能ですね
マグナや神石など編成プリセットを提供できればユーザーからすれば楽そう


召喚石以外のデフォルト値を削除しました

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.

スキル合計でオーガ5% クリュサオル(2刀)20% の表示を確認

@kagekiyo7 kagekiyo7 merged commit 2116275 into MotocalDevelopers:master May 18, 2019
@kagekiyo7 kagekiyo7 deleted the add-default-and-kouzin-jobbonus branch May 18, 2019 03:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants