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.

Sign up for GitHub

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

Jump to bottom

[Mob410] ヘイローン作成 #473

Open
wants to merge 95 commits into
base: master
Choose a base branch
from
Open

[Mob410] ヘイローン作成 #473

wants to merge 95 commits into from
+9,710 −7

Conversation

mokkkk
Copy link
Contributor

@mokkkk mokkkk commented Dec 18, 2024 •
edited
Loading

現時点である程度完成したため、途中までレビューできるようにPRを作成

対象Mob

  • 410

対象オブジェクト

  • 2107
  • 2108
  • 2109
  • 2110
  • 2111
  • 2112
  • 2113
  • 2114
  • 2115
  • 2116
  • 2117
  • 2119
  • 2120
  • 2121
  • 2122
  • 2123
  • 2124
  • 2125
  • 2126
  • 2127
  • 2128
  • 2129

今後の変更予定

  • 大技前の演出の調整
  • 登場、退場演出の追加
  • 意見があれば、行動の調整（難易度変更など）
  • 当たり判定の位置調整（AJ側でやるのでレビューは不要なはず）

mokkkk and others added 30 commits November 25, 2024 21:29
@mokkkk
ヘイローン作成準備
c234887
@mokkkk
汎用処理を記述
906a69e
@mokkkk
実装準備
020be6a
@mokkkk
攻撃用オブジェクト追加
cf2a9b6
@mokkkk
攻撃動作1つ実装
e55eb4d
@mokkkk
動きを暫定実装
aece8ef
@mokkkk
氷技攻撃暫定実装
55f7e9f
@mokkkk
氷技アニメーション演出作成
637e9d0
@mokkkk
氷技仕様変更
31ac46a
@mokkkk
雷攻撃暫定実装
e2ada93
@Lapis-LJA
長方形予告を作成
d30f6f0
@Lapis-LJA
長方形予告の消滅時間を変更
550bc07
@mokkkk
雷の動作調整
138f475
@mokkkk
コンフリクト解消
7f09652
@mokkkk
雷技実装
7160c4f
@mokkkk
タイミングなど調整
a1c2abb
@mokkkk
全属性攻撃暫定実装
29da084
@Lapis-LJA
任意の色を指定できるように変更
24bd810
@Lapis-LJA
ファイル名ミスを修正
e5ac851
@mokkkk
Merge branch 'dev/nukk_mob0410_heiloang' of https://github.com/Projec…
f87cd32 
…tTSB/Asset into dev/nukk_mob0410_heiloang
@mokkkk
複数当たり判定対応
d669550
@mokkkk
全属性攻撃の予兆追加
ee77e1d
@mokkkk
Ehdの処理調整
a8185e1
@mokkkk
移動攻撃大枠作成
db6c2ad
@mokkkk
移動攻撃実装
6614fda
@mokkkk
なぎはらい火炎放射動作実装
03f5946
@mokkkk
演出調整
0bbd740
@mokkkk
竜巻攻撃実装
6e054fe
@mokkkk
落雷演出作成
e0fc821
@mokkkk
雷などの演出調整
3979339
haiiro2gou
haiiro2gou reviewed Jan 2, 2025
View reviewed changes
Copy link
Member

@haiiro2gou haiiro2gou left a comment

Choose a reason for hiding this comment

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

薙ぎ払い

Asset/data/asset/functions/mob/0410.heiloang/tick/event/tornado/.mcfunction Show resolved Hide resolved
Asset/data/asset/functions/object/2122.heiloang_flame_vfx/init/.mcfunction Outdated Show resolved Hide resolved
haiiro2gou
haiiro2gou requested changes Jan 2, 2025
View reviewed changes
Copy link
Member

@haiiro2gou haiiro2gou left a comment

Choose a reason for hiding this comment

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

今回は未実装の 411 と 412 は見てないよ

スキル毎に別の時間にレビューしてる都合取っ散らかってるのは許してほしい
実装上の問題と余裕があれば改善してほしい細かいところのレビューが混ざってるので
気になる点なんかは聞いてくれれば早めに返答する

Asset/data/asset/functions/object/2123.heiloang_flare/load.mcfunction Outdated Show resolved Hide resolved
Asset/data/asset/functions/object/2123.heiloang_flare/init/.mcfunction Outdated Show resolved Hide resolved
Asset/data/asset/functions/object/2123.heiloang_flare/tick/.mcfunction Show resolved Hide resolved
Asset/data/asset/functions/object/2123.heiloang_flare/tick/prediction_normal.mcfunction Show resolved Hide resolved
Asset/data/asset/functions/mob/0410.heiloang/tick/event/plamet/move.mcfunction Outdated Show resolved Hide resolved
Asset/data/asset/functions/mob/0410.heiloang/tick/event/power_breath_continue_1/end.mcfunction Show resolved Hide resolved
Asset/data/asset/functions/object/2128.heiloang_powerbreath/load.mcfunction Outdated Show resolved Hide resolved
Asset/data/asset/functions/object/2128.heiloang_powerbreath/tick/.mcfunction Show resolved Hide resolved
Asset/data/asset/functions/object/2128.heiloang_powerbreath/tick/damage_pursuit.mcfunction Outdated Show resolved Hide resolved
Asset/data/asset/functions/object/2128.heiloang_powerbreath/tick/damage_pursuit.mcfunction Outdated Show resolved Hide resolved
@haiiro2gou
Copy link
Member

AJの当たり判定スライムについて
現状のスライムのエンティティデータだと体力1になっているので
データの編集部分をこう変更しておいてほしい
(頭にアイテムを乗せてmodifierを乗せているが、現在体力の変更処理はその前に行われるため体力１のままになっている)

data merge entity @s {Silent:1b,NoAI:1b,Size:3,DeathLootTable:"minecraft:empty",Attributes:[{Name:"generic.max_health",Base:1024}]}

@mokkkk
Copy link
Contributor Author

mokkkk commented Jan 3, 2025

AJの当たり判定スライムについて 現状のスライムのエンティティデータだと体力1になっているので データの編集部分をこう変更しておいてほしい (頭にアイテムを乗せてmodifierを乗せているが、現在体力の変更処理はその前に行われるため体力１のままになっている)

data merge entity @s {Silent:1b,NoAI:1b,Size:3,DeathLootTable:"minecraft:empty",Attributes:[{Name:"generic.max_health",Base:1024}]}

対応しました

haiiro2gou
haiiro2gou requested changes Jan 3, 2025
View reviewed changes
Asset/data/asset/functions/mob/0410.heiloang/init/.mcfunction Outdated Show resolved Hide resolved
Asset/data/asset/functions/mob/0410.heiloang/tick/act/phase_2/reset_element.mcfunction Outdated Show resolved Hide resolved
Asset/data/asset/functions/mob/0410.heiloang/tick/act/phase_4/reset_element.mcfunction Outdated Show resolved Hide resolved
Asset/data/asset/functions/mob/0410.heiloang/tick/act/phase_1/element.m.mcfunction Show resolved Hide resolved
Asset/data/asset/functions/mob/0410.heiloang/tick/event/richt_blitzen/.mcfunction Outdated Show resolved Hide resolved
Asset/data/asset/functions/object/2117.heiloang_tornado/tick/attack.mcfunction
Comment on lines +21 to +23
# 浮遊エフェクト付与
effect give @a[distance=..30] jump_boost 9 5

Copy link
Member

Choose a reason for hiding this comment

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

そもそもこれ跳躍エフェクトだし
ここで付けるべきなの？という疑問がある

Asset/data/asset/functions/mob/0410.heiloang/tick/event/power_breath_continue_1/end.mcfunction Show resolved Hide resolved
Asset/data/asset/functions/object/2128.heiloang_powerbreath/tick/.mcfunction Show resolved Hide resolved
Asset/data/asset/functions/mob/0410.heiloang/init/locator.mcfunction Outdated Show resolved Hide resolved
@haiiro2gou
Copy link
Member

haiiro2gou commented Jan 3, 2025 •
edited
Loading

あと思ったこととして
死亡時に何もなかったかのように消えていくのは少し気になった

@mokkkk
Copy link
Contributor Author

mokkkk commented Jan 5, 2025

表現があいまいだったので訂正させてもらう
今は7ヶ所に分割して隙間ができるように 4 -> 3 -> 4 の順で攻撃してるわけだけど
3 -> 4 -> 3 のパターンがあってもいいな、という意味だった

対応しました

@mokkkk
Copy link
Contributor Author

mokkkk commented Jan 5, 2025

そもそもこれ跳躍エフェクトだし
ここで付けるべきなの？という疑問がある

設計的な観点なのか、ゲームバランス的な観点の問題なのか分からないので(自分はTSBについてはどちらも疎い)
具体的な解決策というか指針が知りたいです

@mokkkk
Copy link
Contributor Author

mokkkk commented Jan 5, 2025

あと思ったこととして
死亡時に何もなかったかのように消えていくのは少し気になった

登場演出、討伐演出アニメーション作成中です

@haiiro2gou
Copy link
Member

設計的な観点なのか、ゲームバランス的な観点の問題なのか分からないので(自分はTSBについてはどちらも疎い) 具体的な解決策というか指針が知りたいです

竜巻の命中時なので
恐らく浮遊エフェクトを付けるべきところに跳躍エフェクトが付いてるところを指摘した
コメントも実装とかみ合ってないからその辺も併せて

@haiiro2gou
Copy link
Member

それで浮遊エフェクトは
Asset側に送ってるのでEffectAssetにしておくとよい(ID: 125)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@haiiro2gou haiiro2gou haiiro2gou requested changes

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mokkkk @haiiro2gou @Lapis-LJA