Skip to content

Version 2 Beta Test Version #37, #38#43

Open
shun126 wants to merge 2 commits into
mainfrom
37-38-
Open

Version 2 Beta Test Version #37, #38#43
shun126 wants to merge 2 commits into
mainfrom
37-38-

Conversation

@shun126

@shun126 shun126 commented Jun 20, 2026

Copy link
Copy Markdown
Owner

Changes

Migration policy

  • v2.0.0 keeps legacy DeprecatedProperty fields in the runtime class layout so v1 assets can be migrated safely to the v2 parameter groups.
  • FDungeonGeneratedRoomInfo and ADungeonRoomSensorBase::GetGeneratedRoomInfo() are now the standard v2 API for generated room information. Legacy room-information fields on ADungeonRoomSensorBase remain as deprecated migration fields.
  • Removed unreleased v2 room-flag enemy spawn scales from ADungeonRoomSensorBase so helper enemy counts are tuned only by area, GameplayRoleEnemySpawnMultipliers, and StructuralRoleEnemySpawnMultipliers.
  • A future version may limit v1 -> v2 migration to Editor/cook-only flows. After that policy is adopted, directly loading unsaved v1 assets in runtime builds may become unsupported.
  • Legacy properties should only be removed after the project workflow reliably converts and saves v1 assets as v2 assets through the Editor or cook migration path.

変更点

移行ポリシー

  • v2.0.0 では、v1 のアセットを v2 のパラメータグループに安全に移行できるよう、ランタイムクラスのレイアウトにレガシーな DeprecatedProperty フィールドが維持されています。
  • FDungeonGeneratedRoomInfo および ADungeonRoomSensorBase::GetGeneratedRoomInfo() は、生成された部屋情報に関する v2 の標準 API となりました。ADungeonRoomSensorBase 上のレガシーなルーム情報フィールドは、非推奨の移行用フィールドとして残されています。
  • ADungeonRoomSensorBase から、未公開の v2 ルームフラグによる敵スポーン倍率を削除しました。これにより、ヘルパーの敵数は、エリア、GameplayRoleEnemySpawnMultipliers、および StructuralRoleEnemySpawnMultipliers によってのみ調整されるようになりました。
  • 将来のバージョンでは、v1 から v2 への移行がエディタ/クック専用フローに限定される可能性があります。このポリシーが採用された後、ランタイムビルドで保存されていない v1 アセットを直接読み込むことはサポートされなくなる可能性があります。
  • レガシープロパティは、プロジェクトのワークフローにおいて、エディタまたはクックによる移行パスを経由して v1 アセットが確実に v2 アセットに変換・保存された後にのみ削除してください。

@shun126 shun126 self-assigned this Jun 20, 2026
@shun126 shun126 added the enhancement New feature or request label Jun 20, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1c2ca78a87

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/publish-document-to-wiki.yml
*/

#pragma once
#include <BehaviorTree/Tasks/BTTask_MoveTo.h>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Promote AIModule for public BT headers

This new public header includes AIModule's BTTask_MoveTo, while DungeonGenerator.Build.cs still keeps AIModule/GameplayTasks as private dependencies. Any game or plugin module that includes Sample/Task/BTT_DungeonSampleMoveTo.h will not inherit the required AIModule include/link dependency from DungeonGenerator, so consumers can fail to compile; move the dependency to PublicDependencyModuleNames or keep these BT task headers private.

Useful? React with 👍 / 👎.

Comment on lines +73 to +74
if (mMoveTime >= MoveTimeLimit)
FinishLatentTask(OwnerComp, EBTNodeResult::Aborted);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Guard the default move time limit

With MoveTimeLimit defaulting to 0, any task that is not already in the attack-distance band reaches this check after the first tick (DeltaSeconds > 0) and aborts immediately. Sample behavior trees that use the default value, or existing assets that do not override it, will never actually move; treat 0 as disabled or give it a positive default before comparing.

Useful? React with 👍 / 👎.

@shun126 shun126 changed the title Version 2 Alpha Test Version #37, #38 Version 2 Beta Test Version #37, #38 Jun 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant