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

マニュアル追記)新しいオブジェクトが生成される際の条件を記載 #227

Merged
merged 5 commits into from
Jul 26, 2024

Conversation

SYN-miyamoto
Copy link
Contributor

記載場所や内容を確認してください。Unityと同じ内容にしているのでUnrealの単語と会わないものがあればご指示ください

関連リンク

実装内容

マージ前確認項目

  • Squash and Mergeが選択されていること
  • UI、挙動に変更ある場合、ユーザーマニュアルが更新されていること
  • ユニットテストが通っていること

動作確認

その他

記載場所や内容を確認してください。Unityと同じ内容にしているのでUnrealの単語と会わないものがあればご指示ください
@sevendev
Copy link
Contributor

地形変換/高さ合わせ処理の成果物について、ランドスケープ化の成果物は名前の先頭に"DEM_"と付与された名前のランドスケープになります。高さ合わせした地物については、前と同じ名前のコンポーネントが生成されます。

@sevendev
Copy link
Contributor

処理が完了したとき、結果は新しいコンポーネントとして配置されます。その地物のコンポーネント名は地物のGML IDとなります。ただし、主要地物内の同一マテリアル単位で分割結合した場合は該当コンポーネント名は"Material-(マテリアルID)"となります。

コメントを受けて修正しました。ご確認ください
@dazzhe dazzhe merged commit d8cdfc6 into dev/v2 Jul 26, 2024
@dazzhe dazzhe deleted the SYN-miyamoto-patch-4 branch July 26, 2024 14:15
linoal added a commit that referenced this pull request Jul 26, 2024
* Heightmap / Landscape Create (#211)

* LOD4 Import 用 library update

* Heightmap creator

* lib

* lib update

* use meshcode extent debug

* Texture UV

* update

* landscape create from actor

* Param 渡し Refactoring

* OFFSET added.

* Landscape UI

* mac arm lib added.

* 未使用処理削除

* convolution filter
実行時ボタン表記 生成中

* Debug用画像出力フラグ追加

* Material deleted

* Material default value update

* Material save

* Landscape は基本Editorのみ

* ReliefFeatureをフィルタリングしない(静岡用)

* Tile で高速化

* lib update

---------

Co-authored-by: COREI5\taka <seventhdim@gmail.com>

* Landscape fill transparent edges (#213)

* DefualtMaterial NoTexutre Removal (#214)

* LOD4 Import 用 library update

* Default Material No Texture removed

* Material No Texture 削除

---------

Co-authored-by: COREI5\taka <seventhdim@gmail.com>

* Feature/matclassify reconstruct UI merge (#212)

* LOD4 Import 用 library update

* UI 変更

* 処理追加・UI修正

* バグ修正等

* 粒度変更なしの場合の処理追加

* 属性検索bug fix

* asset fix

* cleanup

* ObjectMixerのエディタ設定を変更する処理

* Parameter隠す

* 実行中にボタンを「処理中...」に変更

* PackageTypeSelectWindow reup

* パッケージ選択修正

* 処理実行時に全て非活性化
一部処理中...になっていなかったのを修正

---------

Co-authored-by: COREI5\taka <seventhdim@gmail.com>

* 高さ合わせ (#216)

* LOD4 Import 用 library update

* temp update (build error)

* temp update (crash)

* temp update (軸変換未対応)

* Lib update (座標変換)

* 動作(LOD3 Road以外)

* const value fix

* lib update (mac build)

* lib update (mac crash fixed)

---------

Co-authored-by: COREI5\taka <seventhdim@gmail.com>

* SDKアンケートページへの導線ボタン追加 (#217)

* SDKアンケートページへの導線ボタン追加

* アンケートURLを更新

* モデル調整、地形変換Doc更新 (#218)

* LOD4 Import 用 library update

* Doc更新

* 3D都市モデル編集にマージ

---------

Co-authored-by: COREI5\taka <seventhdim@gmail.com>
Co-authored-by: Kazumasa Sakiyama <kazsakinotwen@gmail.com>

* Update documentation.yml (#220)

* LOD3ROADを地形に合わせる (#219)

* LOD4 Import 用 library update

* temp update (build error)

* temp update (crash)

* temp update (軸変換未対応)

* Lib update (座標変換)

* 動作(LOD3 Road以外)

* const value fix

* lib update (mac build)

* lib update (mac crash fixed)

* LOD3 Road

* Debug処理

* Heightmap書き換え処理変更

* Refactoring

* save aligned image for debug
set results at once

* fix for mac

* cosnt added

* lib update( align invert whiite out fix )

---------

Co-authored-by: COREI5\taka <seventhdim@gmail.com>

* アンケートボタン改善対応 (#223)

* マテリアル分けLib実装 (#221)

* LOD4 Import 用 library update

* temp update (build error)

* temp update (crash)

* temp update (軸変換未対応)

* Lib update (座標変換)

* 動作(LOD3 Road以外)

* const value fix

* lib update (mac build)

* lib update (mac crash fixed)

* LOD3 Road

* Debug処理

* Heightmap書き換え処理変更

* Refactoring

* save aligned image for debug
set results at once

* fix for mac

* cosnt added

* lib update( align invert whiite out fix )

* type/attr convert from model

* comment removed

* comment removed

* gml id children interation

* error fix

* Terrainメッシュ化処理 (#222)

* Terrain Mesh 生成

* icon 変更

* lib update

* mac build fix

* Tab文言変更

---------

Co-authored-by: COREI5\taka <seventhdim@gmail.com>

* 地形変換UIテキスト変更 (#225)

* UI Text変更

* Terrain -> Landscape

* マニュアル追記)モデル調整にUnity同様の追記

* バイナリをリリースビルドに

* マニュアル追記)マテリアル結合の条件を追記 (#229)

マージしますね

* ハイトマップ生成時に頂点座標がずれる旨追記 (#228)

Unityと同じ文章を入れています。Unrealでは、違和感がある場合は正しい内容をご指示ください

* 確認)地形変換高さ合わせについて(Unity同様) (#226)

春名様からのスラックコメント↓下記ですが、こちらの内容が十分に説明されていますか?(宮本判断できずでしたmm) 説明されていたらOKです!

////////
地形モデルの範囲外に及ぶオブジェクト(添付1点目で選択表示になっている土地利用モデル)について、地形モデル範囲外の場合は端の高さを利用して変換をかけていますかね?「交通・区域モデルの高さを地形に合わせる」という表現からは予想できない挙動なので、マニュアルに注記してもらえますか?

恐らく「地形変換」にて「余白を端の高さに合わせる」がONの場合、添付1点目のように別オブジェクトの高さ合わせにも端の高さが適用され、OFFの場合、地形の範囲外は高さ0で統一されると思われます。 ////////

* マニュアル追記)エクスプローラーによるインポートの件を追記 (#230)

* マニュアル追記)エクスプローラーによるインポートの件を追記

Unrealもプロジェクトビューで合ってますか?

* Update Documentation/manual/ExportCityModels.md

---------

Co-authored-by: Kazumasa Sakiyama <kazsakinotwen@gmail.com>

* マニュアル追記)新しいオブジェクトが生成される際の条件を記載 (#227)

* マニュアル追記)新しいオブジェクトが生成される際の条件を記載

記載場所や内容を確認してください。Unityと同じ内容にしているのでUnrealの単語と会わないものがあればご指示ください

* 生成されるオブジェクトの条件や名称を追記

コメントを受けて修正しました。ご確認ください

* Update Documentation/manual/ModelAdjust.md

* Update Documentation/manual/ModelAdjust.md

* TOC更新

---------

Co-authored-by: Kazumasa Sakiyama <kazsakinotwen@gmail.com>

* 同上

---------

Co-authored-by: sevendev <taka@7jigen.net>
Co-authored-by: COREI5\taka <seventhdim@gmail.com>
Co-authored-by: Sacchy <mandw.entertainment@gmail.com>
Co-authored-by: Kazumasa Sakiyama <kazsakinotwen@gmail.com>
Co-authored-by: SYN-miyamoto <145427271+SYN-miyamoto@users.noreply.github.com>
linoal added a commit that referenced this pull request Jul 26, 2024
* Heightmap / Landscape Create (#211)

* LOD4 Import 用 library update

* Heightmap creator

* lib

* lib update

* use meshcode extent debug

* Texture UV

* update

* landscape create from actor

* Param 渡し Refactoring

* OFFSET added.

* Landscape UI

* mac arm lib added.

* 未使用処理削除

* convolution filter
実行時ボタン表記 生成中

* Debug用画像出力フラグ追加

* Material deleted

* Material default value update

* Material save

* Landscape は基本Editorのみ

* ReliefFeatureをフィルタリングしない(静岡用)

* Tile で高速化

* lib update

---------

Co-authored-by: COREI5\taka <seventhdim@gmail.com>

* Landscape fill transparent edges (#213)

* DefualtMaterial NoTexutre Removal (#214)

* LOD4 Import 用 library update

* Default Material No Texture removed

* Material No Texture 削除

---------

Co-authored-by: COREI5\taka <seventhdim@gmail.com>

* Feature/matclassify reconstruct UI merge (#212)

* LOD4 Import 用 library update

* UI 変更

* 処理追加・UI修正

* バグ修正等

* 粒度変更なしの場合の処理追加

* 属性検索bug fix

* asset fix

* cleanup

* ObjectMixerのエディタ設定を変更する処理

* Parameter隠す

* 実行中にボタンを「処理中...」に変更

* PackageTypeSelectWindow reup

* パッケージ選択修正

* 処理実行時に全て非活性化
一部処理中...になっていなかったのを修正

---------

Co-authored-by: COREI5\taka <seventhdim@gmail.com>

* 高さ合わせ (#216)

* LOD4 Import 用 library update

* temp update (build error)

* temp update (crash)

* temp update (軸変換未対応)

* Lib update (座標変換)

* 動作(LOD3 Road以外)

* const value fix

* lib update (mac build)

* lib update (mac crash fixed)

---------

Co-authored-by: COREI5\taka <seventhdim@gmail.com>

* SDKアンケートページへの導線ボタン追加 (#217)

* SDKアンケートページへの導線ボタン追加

* アンケートURLを更新

* モデル調整、地形変換Doc更新 (#218)

* LOD4 Import 用 library update

* Doc更新

* 3D都市モデル編集にマージ

---------

Co-authored-by: COREI5\taka <seventhdim@gmail.com>
Co-authored-by: Kazumasa Sakiyama <kazsakinotwen@gmail.com>

* Update documentation.yml (#220)

* LOD3ROADを地形に合わせる (#219)

* LOD4 Import 用 library update

* temp update (build error)

* temp update (crash)

* temp update (軸変換未対応)

* Lib update (座標変換)

* 動作(LOD3 Road以外)

* const value fix

* lib update (mac build)

* lib update (mac crash fixed)

* LOD3 Road

* Debug処理

* Heightmap書き換え処理変更

* Refactoring

* save aligned image for debug
set results at once

* fix for mac

* cosnt added

* lib update( align invert whiite out fix )

---------

Co-authored-by: COREI5\taka <seventhdim@gmail.com>

* アンケートボタン改善対応 (#223)

* マテリアル分けLib実装 (#221)

* LOD4 Import 用 library update

* temp update (build error)

* temp update (crash)

* temp update (軸変換未対応)

* Lib update (座標変換)

* 動作(LOD3 Road以外)

* const value fix

* lib update (mac build)

* lib update (mac crash fixed)

* LOD3 Road

* Debug処理

* Heightmap書き換え処理変更

* Refactoring

* save aligned image for debug
set results at once

* fix for mac

* cosnt added

* lib update( align invert whiite out fix )

* type/attr convert from model

* comment removed

* comment removed

* gml id children interation

* error fix

* Terrainメッシュ化処理 (#222)

* Terrain Mesh 生成

* icon 変更

* lib update

* mac build fix

* Tab文言変更

---------

Co-authored-by: COREI5\taka <seventhdim@gmail.com>

* 地形変換UIテキスト変更 (#225)

* UI Text変更

* Terrain -> Landscape

* マニュアル追記)モデル調整にUnity同様の追記

* バイナリをリリースビルドに

* マニュアル追記)マテリアル結合の条件を追記 (#229)

マージしますね

* ハイトマップ生成時に頂点座標がずれる旨追記 (#228)

Unityと同じ文章を入れています。Unrealでは、違和感がある場合は正しい内容をご指示ください

* 確認)地形変換高さ合わせについて(Unity同様) (#226)

春名様からのスラックコメント↓下記ですが、こちらの内容が十分に説明されていますか?(宮本判断できずでしたmm) 説明されていたらOKです!

////////
地形モデルの範囲外に及ぶオブジェクト(添付1点目で選択表示になっている土地利用モデル)について、地形モデル範囲外の場合は端の高さを利用して変換をかけていますかね?「交通・区域モデルの高さを地形に合わせる」という表現からは予想できない挙動なので、マニュアルに注記してもらえますか?

恐らく「地形変換」にて「余白を端の高さに合わせる」がONの場合、添付1点目のように別オブジェクトの高さ合わせにも端の高さが適用され、OFFの場合、地形の範囲外は高さ0で統一されると思われます。 ////////

* マニュアル追記)エクスプローラーによるインポートの件を追記 (#230)

* マニュアル追記)エクスプローラーによるインポートの件を追記

Unrealもプロジェクトビューで合ってますか?

* Update Documentation/manual/ExportCityModels.md

---------

Co-authored-by: Kazumasa Sakiyama <kazsakinotwen@gmail.com>

* マニュアル追記)新しいオブジェクトが生成される際の条件を記載 (#227)

* マニュアル追記)新しいオブジェクトが生成される際の条件を記載

記載場所や内容を確認してください。Unityと同じ内容にしているのでUnrealの単語と会わないものがあればご指示ください

* 生成されるオブジェクトの条件や名称を追記

コメントを受けて修正しました。ご確認ください

* Update Documentation/manual/ModelAdjust.md

* Update Documentation/manual/ModelAdjust.md

* TOC更新

---------

Co-authored-by: Kazumasa Sakiyama <kazsakinotwen@gmail.com>

* 同上

---------

Co-authored-by: sevendev <taka@7jigen.net>
Co-authored-by: COREI5\taka <seventhdim@gmail.com>
Co-authored-by: Sacchy <mandw.entertainment@gmail.com>
Co-authored-by: Kazumasa Sakiyama <kazsakinotwen@gmail.com>
Co-authored-by: SYN-miyamoto <145427271+SYN-miyamoto@users.noreply.github.com>
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