Skip to content

[Artifact 1598-1599] 堕天の作成 - #2116

Open
firgde wants to merge 33 commits into
masterfrom
dev/firgde_artifact1598-1599
Open

firgde wants to merge 33 commits into
masterfrom
dev/firgde_artifact1598-1599

Conversation

@firgde

@firgde firgde commented Sep 8, 2026 •

Copy link
Copy Markdown
Member

以下のAssetを追加

  • Artifact 1598 堕天
  • Artifact 1599 堕天
  • Object 1188 堕天・残留攻撃

@firgde
firgde marked this pull request as ready for review September 16, 2026 19:22

@tatuharu tatuharu left a comment •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

このレビューはハルが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 tatuharu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

このレビューはハルが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]

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

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 を変えます。

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

修正を確認しました。輪の展開アニメーションが消える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 と同じ時点です。

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.

3 participants