Repository navigation
Conversation
There was a problem hiding this comment.
このレビューはハルがAI自動レビューの動作テストのためにOpus5.5 Extra Highを使用してレビューを行なったものです。鵜呑みにせず、指摘を採用するかはお任せします。
指摘が 3 件あります。1件目と2件目の詳細は行コメントに、3件目は一覧の下に書いています。
- 〔不具合〕輪の攻撃で
ElementTypeが空文字になり、魔法の補正が二重にかかる - 〔不具合〕ダメージを与えなかった個体に
1598.Targetが残る - 〔契約〕1598 の register がなく、レアリティから神器を選ぶ処理の対象にならない(下に説明)
1598 には register.mcfunction がなく、tags/functions/artifact/register.json にも追加されていません。このままでは RarityRegistry に登録されず、api:artifact/*/from_rarity でレアリティから神器を選ぶ処理の対象になりません。神器プールから入手できるようにする予定であれば、追加が必要です。
tatuharu
left a comment
There was a problem hiding this comment.
このレビューはハルがAI自動レビューの動作テストのためにOpus5.5 Extra Highを使用してレビューを行なったものです。鵜呑みにせず、指摘を採用するかはお任せします。
指摘が 1 件あります。詳細は行コメントに書いています。
- 〔不具合〕輪の展開アニメーションが、召喚の3tick後ではなく消える3tick前に始まる
| scoreboard players remove @s General.Object.Tick 1 | ||
|
|
||
| # 最初のtickだけ展開アニメーション | ||
| execute if score @s General.Object.Tick matches 3 run data modify entity @s transformation.scale set value [4.5f,4.5f,0.0f] |
There was a problem hiding this comment.
1f2e7e9 で、General.Object.Tick に init で Tick(神器から渡す160)を入れ、毎tick減らす形に変わりました。一方で11行目の展開アニメーションの条件は matches 3 のままです。以前は召喚の3tick後に scale を4.5にしていましたが、いまは残りが3tickになったとき(召喚の157tick後)に変わります。召喚時の scale は0なので、輪は消える直前の数tickしか見えません。攻撃の間隔と、18行目で消える時点は以前と同じです。
検証用サーバー(Minecraft 1.20.4、このPRの head 017bea7)で、damage/boss.mcfunction と同じ FieldOverride(Tick 160、AttackInterval 20 など)で輪を召喚し、transformation.scale を読みました。scale は召喚の156tick後まで [0.0f, 0.0f, 0.0f] のままで、157tick後に [4.5f, 4.5f, 0.0f] になり、160tick後に輪が消えています。bb35b33 で同じ手順を流すと、scale は召喚の3tick後に [4.5f, 4.5f, 0.0f] になりました。クライアントでの描画は確かめていません。
召喚からの経過tickで判定すれば、以前と同じ時点に戻ります。例えば、this.Tick から General.Object.Tick を引いた値が3のときに scale を変えます。
There was a problem hiding this comment.
修正を確認しました。輪の展開アニメーションが消える3tick前に始まる件は解消されていて、この指摘への対応として問題ありません。
087295b で、init が O1188.InterpolationTick に存在時間から3を引いた値を入れ、11行目はこの値と General.Object.Tick が等しくなったときに scale を変えるようになりました。存在時間が160なら、召喚の3tick後に変わります。
検証用サーバー(Minecraft 1.20.4、このPRの head 087295b)で、前回と同じ手順で確かめました。scale は召喚の3tick後に [4.5f, 4.5f, 0.0f] になり、その後は消える160tick後までそのままでした。bb35b33 と同じ時点です。
以下のAssetを追加