マテリアル分けLib実装 #221
Merged
マテリアル分けLib実装 #221
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…for-Unreal into feature/align_height_lod3road # Conflicts: # Content/EUW/ModelLandscapeTab.uasset # Source/PLATEAURuntime/Private/PLATEAUInstancedCityModel.cpp # Source/PLATEAURuntime/Private/Reconstruct/PLATEAUModelAlignLand.cpp # Source/PLATEAURuntime/Public/PLATEAUInstancedCityModel.h # Source/PLATEAURuntime/Public/Reconstruct/PLATEAUModelAlignLand.h
set results at once
…for-Unreal into feature/align_height_lod3road
…for-Unreal into feature/reconstruct_by_material_based_lod3road # Conflicts: # Source/ThirdParty/lib/windows/plateau_combined.lib
dazzhe
reviewed
Jul 24, 2024
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
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
関連リンク
実装内容
libのAdjasterを使用してType, 属性のマテリアル分け
粒度で、マテリアルごとの分割を実装
マージ前確認項目
動作確認
マテリアルごとの分割が行えること
Type, 属性のマテリアル分けが行えること
その他
■既知のバグ
・BuildingInstallation のマテリアル分けが行われない
・何度か粒度変更を繰り返すとテクスチャの参照が消えてしまう