exclude_triangles_outside_extentがUV4に反映されない問題の修正 (Synesthesias#176)

* exclude_outside_trianglesがUV4に反映されない問題の修正

* バグ修正

Author: dazzhe

include/plateau/polygon_mesh/city_object_list.h

@@ -11,6 +11,16 @@ namespace plateau::polygonMesh {
         int primary_index;
         int atomic_index;
 
+        CityObjectIndex()
+            : primary_index(0)
+            , atomic_index(0) {
+        }
+
+        CityObjectIndex(const int primary_index, const int atomic_index)
+            : primary_index(primary_index)
+            , atomic_index(atomic_index) {
+        }
+
         static CityObjectIndex fromUV(const TVec2f& uv) {
             return {
                 static_cast<int>(std::lround(uv.x)),

include/plateau/polygon_mesh/mesh_extract_options.h

@@ -32,7 +32,7 @@ namespace plateau::polygonMesh {
                 unit_scale(1.0),
                 coordinate_zone_id(9), // 東京で歪みの少ない直交座標系をデフォルトとします。
                 exclude_city_object_outside_extent(true),
-                exclude_triangles_outside_extent(false),
+                exclude_polygons_outside_extent(false),
                 extent(geometry::Extent::all()) // 全範囲をデフォルトとします。
                 {}
 
@@ -71,7 +71,7 @@ namespace plateau::polygonMesh {
          * その方法とは、メッシュ操作によって、範囲外に存在するポリゴンを除外します。
          * この方法であれば 10km×10km の地形など巨大なオブジェクトにも対応できます。
          */
-        bool exclude_triangles_outside_extent;
+        bool exclude_polygons_outside_extent;
 
         geometry::Extent extent;
     };

include/plateau/polygon_mesh/mesh_factory.h

@@ -0,0 +1,91 @@
+#pragma once
+
+#include <plateau/polygon_mesh/mesh.h>
+#include <plateau/polygon_mesh/mesh_extract_options.h>
+#include <citygml/polygon.h>
+#include <citygml/cityobject.h>
+#include <list>
+#include "plateau/geometry/geo_reference.h"
+
+namespace plateau::polygonMesh {
+
+    /**
+     * citygml::Polygon(CityGMLパーサーの出力)をpolygonMesh::Mesh(3Dモデルデータ)に変換する機能を提供します。
+     */
+    class LIBPLATEAU_EXPORT MeshFactory {
+    public:
+        MeshFactory(
+            std::unique_ptr<Mesh>&& target = nullptr,
+            const MeshExtractOptions& mesh_extract_options = MeshExtractOptions(),
+            const geometry::GeoReference& geo_reference = geometry::GeoReference(9));
+
+        std::unique_ptr<Mesh> releaseMesh() {
+            return std::move(mesh_);
+        }
+
+        /**
+         * citygml::Polygon の情報を Mesh 向けに変換し、 引数の mesh に書き加えます。
+         * 引数で与えられたポリゴンのうち、次の情報を追加します。
+         * ・頂点リスト、インデックスリスト、UV1、テクスチャ。
+         * なおその他の情報のマージには未対応です。例えば LinearRing は考慮されません。
+         * options.export_appearance の値によって、 mergeWithTexture または mergeWithoutTexture を呼び出します。
+         */
+        void addPolygon(const citygml::Polygon& polygon, const std::string& gml_path) const;
+        
+        /**
+         * 主要地物の主要地物IDを設定しMeshをマージします。
+         */
+        void addPolygonsInPrimaryCityObject(
+            const citygml::CityObject& city_object, unsigned lod,
+            const std::string& gml_path);
+
+        /**
+         * 最小地物のMeshのuv4フィールドに最小地物IDを設定するために、最小地物を構成するPolygonに含まれるVertex数を取得します。
+         */
+        static long long countVertices(const citygml::CityObject& city_object, unsigned int lod);
+
+        /**
+         * 最小地物に含まれるすべてのポリゴンをメッシュに追加します。
+         */
+        void addPolygonsInAtomicCityObject(
+            const citygml::CityObject& parent_city_object,
+            const citygml::CityObject& city_object,
+            const unsigned lod, const std::string& gml_path);
+
+        /**
+         * 最小地物に含まれるすべてのポリゴンをメッシュに追加します。
+         */
+        void addPolygonsInAtomicCityObjects(
+            const citygml::CityObject& parent_city_object,
+            const std::list<const citygml::CityObject*>& city_objects,
+            unsigned lod, const std::string& gml_path);
+
+        /**
+         * city_obj に含まれるポリゴンをすべて検索し、リストで返します。
+         * 子の CityObject は検索しません。
+         * 子の Geometry は再帰的に検索します。
+         */
+        static void findAllPolygons(const citygml::CityObject& city_obj, unsigned lod, std::list<const citygml::Polygon*>& out_polygons, long long& out_vertices_count, const plateau::geometry::Extent& extent = plateau::geometry::Extent::all());
+
+        /**
+         * PLATEAUからメッシュを読み込んで座標軸を変換をするとき、このままだとメッシュが裏返ることがあります（座標軸が反転したりするので）。
+         * 裏返りを補正する必要があるかどうかを bool で返します。
+         */
+        static bool shouldInvertIndicesOnMeshConvert(plateau::geometry::CoordinateSystem sys);
+    private:
+        MeshExtractOptions options_;
+        geometry::GeoReference geo_reference_;
+
+        std::unique_ptr<Mesh> mesh_;
+        // 新規に主要地物を追加する際に利用可能なインデックス
+        CityObjectIndex available_primary_index_;
+        // 主要地物のgml::idに対して子(最小地物)を追加する際に利用可能なインデックス
+        std::map<std::string, CityObjectIndex> available_atomic_indices_;
+
+        // 最後に個別に追加された最小地物のインデックスとその親のgml:idのキャッシュ(高速化用)
+        CityObjectIndex last_atomic_index_cache_;
+        std::string last_parent_gml_id_cache_;
+
+        CityObjectIndex createAvailableAtomicIndex(const std::string& parent_gml_id);
+    };
+}

include/plateau/polygon_mesh/mesh_merger.h

@@ -1,130 +1,18 @@
 #pragma once
 
 #include <plateau/polygon_mesh/mesh.h>
-#include <plateau/polygon_mesh/mesh_extract_options.h>
-#include <citygml/polygon.h>
-#include <citygml/cityobject.h>
-#include <list>
-#include "plateau/geometry/geo_reference.h"
 
 namespace plateau::polygonMesh {
 
     /**
-     * citygml::Polygon(CityGMLパーサーの出力)をpolygonMesh::Mesh(3Dモデルデータ)に変換する機能を提供します。
+     * Meshを結合する機能を提供します。
      */
-    class LIBPLATEAU_EXPORT MeshFactory {
+    class LIBPLATEAU_EXPORT MeshMerger {
     public:
-        MeshFactory(
-            std::unique_ptr<Mesh>&& target = nullptr,
-            const MeshExtractOptions& mesh_extract_options = MeshExtractOptions(),
-            const geometry::GeoReference& geo_reference = geometry::GeoReference(9));
-
-        std::unique_ptr<Mesh> releaseMesh() {
-            return std::move(mesh_);
-        }
-
-        /**
-         * citygml::Polygon の情報を Mesh 向けに変換し、 引数の mesh に書き加えます。
-         * 引数で与えられたポリゴンのうち、次の情報を追加します。
-         * ・頂点リスト、インデックスリスト、UV1、テクスチャ。
-         * なおその他の情報のマージには未対応です。例えば LinearRing は考慮されません。
-         * options.export_appearance の値によって、 mergeWithTexture または mergeWithoutTexture を呼び出します。
-         */
-        void addPolygon(const citygml::Polygon& polygon, const std::string& gml_path);
-
-        /**
-         * Mesh に Polygon をマージする代わりに Mesh をマージする版です。
-         */
-        void mergeMesh(const Mesh& other_mesh, bool invert_mesh_front_back, bool include_textures) const;
-
-        /**
-         * Mesh に Polygon をマージする代わりに、データ配列を直接 move で渡す版です。
-         */
-        void mergeMeshInfo(
-            std::vector<TVec3d>&& vertices, std::vector<unsigned>&& indices,
-            UV&& uv_1, std::vector<SubMesh>&& sub_meshes,
-            plateau::geometry::CoordinateSystem mesh_axis_convert_from,
-            plateau::geometry::CoordinateSystem mesh_axis_convert_to, bool include_texture);
-
-        /**
-         * 主要地物の主要地物IDを設定しMeshをマージします。
-         */
-        void addPolygonsInPrimaryCityObject(
-            const citygml::CityObject& city_object, unsigned lod,
-            const std::string& gml_path);
-
-        /**
-         * 最小地物のMeshのuv4フィールドに最小地物IDを設定するために、最小地物を構成するPolygonに含まれるVertex数を取得します。
-         */
-        static long long countVertices(const citygml::CityObject& city_object, unsigned int lod);
-
-        /**
-         * 最小地物に含まれるすべてのポリゴンをメッシュに追加します。
-         */
-        void addPolygonsInAtomicCityObject(
-            const citygml::CityObject& parent_city_object,
-            const citygml::CityObject& city_object,
-            const unsigned lod, const std::string& gml_path);
-
         /**
-         * 最小地物に含まれるすべてのポリゴンをメッシュに追加します。
+         * Meshをマージします。
          */
-        void addPolygonsInAtomicCityObjects(
-            const citygml::CityObject& parent_city_object,
-            const std::list<const citygml::CityObject*>& city_objects,
-            unsigned lod, const std::string& gml_path);
-
-        /**
-         * city_obj に含まれるポリゴンをすべて検索し、リストで返します。
-         * 子の CityObject は検索しません。
-         * 子の Geometry は再帰的に検索します。
-         */
-        static void findAllPolygons(const citygml::CityObject& city_obj, unsigned lod, std::list<const citygml::Polygon*>& out_polygons, long long& out_vertices_count);
-
-        /**
-         * PLATEAUからメッシュを読み込んで座標軸を変換をするとき、このままだとメッシュが裏返ることがあります（座標軸が反転したりするので）。
-         * 裏返りを補正する必要があるかどうかを bool で返します。
-         */
-        static bool shouldInvertIndicesOnMeshConvert(plateau::geometry::CoordinateSystem sys);
-    private:
-        MeshExtractOptions options_;
-        geometry::GeoReference geo_reference_;
-
-        std::unique_ptr<Mesh> mesh_;
-        // 新規に主要地物を追加する際に利用可能なインデックス
-        CityObjectIndex available_primary_index_;
-        // 主要地物のgml::idに対して子(最小地物)を追加する際に利用可能なインデックス
-        std::map<std::string, CityObjectIndex> available_atomic_indices_;
-
-        // 最後に個別に追加された最小地物のインデックスとその親のgml:idのキャッシュ(高速化用)
-        CityObjectIndex last_atomic_index_cache_;
-        std::string last_parent_gml_id_cache_;
-
-        /**
-         * merge関数 のテクスチャ無し版です。
-         * 生成される Mesh の SubMesh はただ1つであり、そのテクスチャパスは空文字列となります。
-         */
-        void mergeWithoutTexture(const Mesh& other_mesh, bool invert_mesh_front_back) const;
-
-        /**
-         * merge関数 のテクスチャあり版です。
-         * テクスチャについては、マージした結果、範囲とテクスチャを対応付ける SubMesh が追加されます。
-         */
-        void mergeWithTexture(const Mesh& other_mesh, bool invert_mesh_front_back) const;
-
-        /**
-         * 形状情報をマージします。merge関数における SubMesh を扱わない版です。
-         */
-        void mergeShape(const Mesh& other_mesh, const bool invert_mesh_front_back) const {
-            const auto prev_num_vertices = mesh_->getVertices().size();
-            const auto other_num_vertices = other_mesh.getVertices().size();
-
-            mesh_->addVerticesList(other_mesh.getVertices());
-            mesh_->addIndicesList(other_mesh.getIndices(), prev_num_vertices, invert_mesh_front_back);
-            mesh_->addUV1(other_mesh.getUV1(), static_cast<unsigned>(other_num_vertices));
-            mesh_->addUV4(other_mesh.getUV4(), static_cast<unsigned>(other_num_vertices));
-        }
-
-        CityObjectIndex createAvailableAtomicIndex(const std::string& parent_gml_id);
+        static void mergeMesh(
+            Mesh& mesh, const Mesh& other_mesh, bool invert_mesh_front_back, bool include_textures);
     };
 }