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

Terrainメッシュ化処理 #222

Conversation

sevendev
Copy link
Contributor

@sevendev sevendev commented Jul 24, 2024

関連リンク

Feature/hmmの処理共通化 #250

実装内容

Terrainメッシュ化処理

マージ前確認項目

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

動作確認

地形変換で、ランドスケープ作成のチェックを入れない場合、平滑化メッシュに変換

その他

モデル調整のボタンアセット変更含む

Copy link
Contributor

@dazzhe dazzhe left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@sevendev sevendev merged commit 6a11444 into feature/reconstruct_by_material_based_lod3road Jul 25, 2024
@sevendev sevendev deleted the feature/landscape_mesh_with_lib branch July 25, 2024 04:08
sevendev added a commit that referenced this pull request Jul 25, 2024
* 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>
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.

2 participants