Transform landmark to a class

src/aliceVision/depthMap/SgmDepthList.cpp

@@ -298,10 +298,10 @@ void SgmDepthList::getMinMaxMidNbDepthFromSfM(float& out_min, float& out_max, fl
         const Point3d point(landmark.X(0), landmark.X(1), landmark.X(2));
 
         // find rc observation
-        const auto it = landmark.observations.find(viewId);
+        const auto it = landmark.getObservations().find(viewId);
 
         // no rc observation
-        if (it == landmark.observations.end())
+        if (it == landmark.getObservations().end())
             continue;
 
         // get rc 2d observation
@@ -370,14 +370,14 @@ void SgmDepthList::getRcTcDepthRangeFromSfM(int tc, double& out_zmin, double& ou
         const Point3d point(landmark.X(0), landmark.X(1), landmark.X(2));
 
         // no tc observation
-        if (landmark.observations.find(tcViewId) == landmark.observations.end())
+        if (landmark.getObservations().find(tcViewId) == landmark.getObservations().end())
             continue;
 
         // find rc observation
-        const auto it = landmark.observations.find(rcViewId);
+        const auto it = landmark.getObservations().find(rcViewId);
 
         // no rc observation
-        if (it == landmark.observations.end())
+        if (it == landmark.getObservations().end())
             continue;
 
         // get rc 2d observation

src/aliceVision/depthMap/volumeIO.cpp

@@ -183,7 +183,7 @@ void exportSimilarityVolume(const CudaHostMemoryHeap<TSim, 3>& in_volumeSim_hmh,
                     continue;
                 const rgb c = getRGBFromJetColorMap(simValue / maxValue);
                 pointCloud.getLandmarks()[landmarkId] = sfmData::Landmark(
-                  Vec3(p.x, p.y, p.z), feature::EImageDescriberType::UNKNOWN, sfmData::Observations(), image::RGBColor(c.r, c.g, c.b));
+                  Vec3(p.x, p.y, p.z), feature::EImageDescriberType::UNKNOWN, image::RGBColor(c.r, c.g, c.b));
 
                 ++landmarkId;
             }
@@ -235,7 +235,7 @@ void exportSimilarityVolumeCross(const CudaHostMemoryHeap<TSim, 3>& in_volumeSim
 
                 const rgb c = getRGBFromJetColorMap(simValue / maxValue);
                 pointCloud.getLandmarks()[landmarkId] = sfmData::Landmark(
-                  Vec3(p.x, p.y, p.z), feature::EImageDescriberType::UNKNOWN, sfmData::Observations(), image::RGBColor(c.r, c.g, c.b));
+                  Vec3(p.x, p.y, p.z), feature::EImageDescriberType::UNKNOWN, image::RGBColor(c.r, c.g, c.b));
 
                 ++landmarkId;
             }
@@ -293,7 +293,7 @@ void exportSimilarityVolumeCross(const CudaHostMemoryHeap<TSimRefine, 3>& in_vol
 
                 const rgb c = getRGBFromJetColorMap(simValue / maxValue);
                 pointCloud.getLandmarks()[landmarkId] = sfmData::Landmark(
-                  Vec3(p.x, p.y, p.z), feature::EImageDescriberType::UNKNOWN, sfmData::Observations(), image::RGBColor(c.r, c.g, c.b));
+                  Vec3(p.x, p.y, p.z), feature::EImageDescriberType::UNKNOWN, image::RGBColor(c.r, c.g, c.b));
 
                 ++landmarkId;
             }
@@ -365,7 +365,7 @@ void exportSimilarityVolumeTopographicCut(const CudaHostMemoryHeap<TSim, 3>& in_
 
             const rgb c = getRGBFromJetColorMap(simValueNorm);
             pointCloud.getLandmarks()[landmarkId] =
-              sfmData::Landmark(Vec3(p.x, p.y, p.z), feature::EImageDescriberType::UNKNOWN, sfmData::Observations(), image::RGBColor(c.r, c.g, c.b));
+              sfmData::Landmark(Vec3(p.x, p.y, p.z), feature::EImageDescriberType::UNKNOWN, image::RGBColor(c.r, c.g, c.b));
 
             ++landmarkId;
         }
@@ -430,7 +430,7 @@ void exportSimilarityVolumeTopographicCut(const CudaHostMemoryHeap<TSimRefine, 3
 
             const rgb c = getRGBFromJetColorMap(simValueColor);
             pointCloud.getLandmarks()[landmarkId] =
-              sfmData::Landmark(Vec3(p.x, p.y, p.z), feature::EImageDescriberType::UNKNOWN, sfmData::Observations(), image::RGBColor(c.r, c.g, c.b));
+              sfmData::Landmark(Vec3(p.x, p.y, p.z), feature::EImageDescriberType::UNKNOWN, image::RGBColor(c.r, c.g, c.b));
 
             ++landmarkId;
         }
@@ -491,7 +491,7 @@ void exportColorVolume(const CudaHostMemoryHeap<float4, 3>& in_volumeSim_hmh,
                 float4 colorValue = *get3DBufferAt_h<float4>(in_volumeSim_hmh.getBuffer(), spitch, pitch, vx, vy, vz);
                 const rgb c = float4_to_rgb(colorValue);  // TODO: convert Lab color into sRGB color
                 pointCloud.getLandmarks()[landmarkId] = sfmData::Landmark(
-                  Vec3(p.x, p.y, p.z), feature::EImageDescriberType::UNKNOWN, sfmData::Observations(), image::RGBColor(c.r, c.g, c.b));
+                  Vec3(p.x, p.y, p.z), feature::EImageDescriberType::UNKNOWN, image::RGBColor(c.r, c.g, c.b));
 
                 ++landmarkId;
             }

src/aliceVision/fuseCut/DelaunayGraphCut.cpp

@@ -775,10 +775,10 @@ void DelaunayGraphCut::addPointsFromSfM(const Point3d hexah[8], const StaticVect
         {
             *vCoordsIt = p;
 
-            vAttrIt->nrc = landmark.observations.size();
+            vAttrIt->nrc = landmark.getObservations().size();
             vAttrIt->cams.reserve(vAttrIt->nrc);
 
-            for (const auto& observationPair : landmark.observations)
+            for (const auto& observationPair : landmark.getObservations())
                 vAttrIt->cams.push_back(_mp.getIndexFromViewId(observationPair.first));
 
             vAttrIt->pixSize = _mp.getCamsMinPixelSize(p, vAttrIt->cams);

src/aliceVision/fuseCut/DelaunayGraphCut_test.cpp

@@ -200,7 +200,7 @@ SfMData generateSfm(const NViewDatasetConfigurator& config, const size_t size, c
         for (int j = 0; j < camsPts.size(); ++j)
         {
             const Vec2 pt = projectedPtsPerCam[j].col(i);
-            landmark.observations[j] = Observation(pt, i, unknownScale);
+            landmark.getObservations()[j] = Observation(pt, i, unknownScale);
         }
         sfm_data.getLandmarks()[i] = landmark;
     }

src/aliceVision/fuseCut/Fuser.cpp

@@ -360,7 +360,7 @@ float Fuser::computeAveragePixelSizeInHexahedron(Point3d* hexah, const sfmData::
         const sfmData::Landmark& landmark = landmarkPair.second;
         const Point3d p(landmark.X(0), landmark.X(1), landmark.X(2));
 
-        for (const auto& observationPair : landmark.observations)
+        for (const auto& observationPair : landmark.getObservations())
         {
             const IndexT viewId = observationPair.first;
 
@@ -506,14 +506,14 @@ void Fuser::divideSpaceFromDepthMaps(Point3d* hexah, float& minPixSize)
 
 bool checkLandmarkMinObservationAngle(const sfmData::SfMData& sfmData, const sfmData::Landmark& landmark, float minObservationAngle)
 {
-    for (const auto& observationPairI : landmark.observations)
+    for (const auto& observationPairI : landmark.getObservations())
     {
         const IndexT I = observationPairI.first;
         const sfmData::View& viewI = *(sfmData.getViews().at(I));
         const geometry::Pose3 poseI = sfmData.getPose(viewI).getTransform();
         const camera::IntrinsicBase* intrinsicPtrI = sfmData.getIntrinsicPtr(viewI.getIntrinsicId());
 
-        for (const auto& observationPairJ : landmark.observations)
+        for (const auto& observationPairJ : landmark.getObservations())
         {
             const IndexT J = observationPairJ.first;
 
@@ -562,7 +562,7 @@ void Fuser::divideSpaceFromSfM(const sfmData::SfMData& sfmData, Point3d* hexah,
         const sfmData::Landmark& landmark = landmarkPair.second;
 
         // check number of observations
-        if (landmark.observations.size() < minObservations)
+        if (landmark.getObservations().size() < minObservations)
             continue;
 
         // check angle between observations

src/aliceVision/localization/CCTagLocalizer.cpp

@@ -77,7 +77,7 @@ bool CCTagLocalizer::loadReconstructionDescriptors(const sfmData::SfMData& sfm_d
         IndexT trackId = landmarkValue.first;
         const sfmData::Landmark& landmark = landmarkValue.second;
 
-        for (const auto& obs : landmark.observations)
+        for (const auto& obs : landmark.getObservations())
         {
             const IndexT viewId = obs.first;
             const sfmData::Observation& obs2d = obs.second;

src/aliceVision/localization/VoctreeLocalizer.cpp

@@ -286,7 +286,7 @@ bool VoctreeLocalizer::initDatabase(const std::string& vocTreeFilepath, const st
         IndexT trackId = landmarkValue.first;
         const sfmData::Landmark& landmark = landmarkValue.second;
 
-        for (const auto& obs : landmark.observations)
+        for (const auto& obs : landmark.getObservations())
         {
             const IndexT viewId = obs.first;
             const sfmData::Observation& obs2d = obs.second;

src/aliceVision/localization/optimization.cpp

@@ -148,11 +148,11 @@ bool refineSequence(std::vector<LocalizationResult>& vec_localizationResult,
                 assert(landmark.descType == match.descType);
                 // normally there should be no other features already associated to this
                 // 3D point in this view
-                if (landmark.observations.count(viewID) != 0)
+                if (landmark.getObservations().count(viewID) != 0)
                 {
                     // this is weird but it could happen when two features are really close to each other (?)
                     ALICEVISION_LOG_DEBUG("Point 3D " << match.landmarkId << " has multiple features "
-                                                      << "in the same view " << viewID << ", current size of obs: " << landmark.observations.size());
+                                                      << "in the same view " << viewID << ", current size of obs: " << landmark.getObservations().size());
                     ALICEVISION_LOG_DEBUG("its associated features are: ");
                     for (std::size_t i = 0; i < matches.size(); ++i)
                     {
@@ -169,15 +169,15 @@ bool refineSequence(std::vector<LocalizationResult>& vec_localizationResult,
                 }
 
                 // the 3D point exists already, add the observation
-                landmark.observations[viewID] = sfmData::Observation(feature, match.featId, unknownScale);
+                landmark.getObservations()[viewID] = sfmData::Observation(feature, match.featId, unknownScale);
             }
             else
             {
                 // create a new 3D point
                 sfmData::Landmark newLandmark;
                 newLandmark.descType = match.descType;
                 newLandmark.X = currResult.getPt3D().col(idx);
-                newLandmark.observations[viewID] = sfmData::Observation(feature, match.featId, unknownScale);
+                newLandmark.getObservations()[viewID] = sfmData::Observation(feature, match.featId, unknownScale);
                 tinyScene.getLandmarks()[match.landmarkId] = std::move(newLandmark);
             }
         }
@@ -188,7 +188,7 @@ bool refineSequence(std::vector<LocalizationResult>& vec_localizationResult,
     //    sfmData::Landmarks &landmarks = tinyScene.getLandmarks();
     //    for(sfmData::Landmarks::iterator it = landmarks.begin(), ite = landmarks.end(); it != ite;)
     //    {
-    //      if(it->second.observations.size() < 5)
+    //      if(it->second.getObservations().size() < 5)
     //      {
     //         it = landmarks.erase(it);
     //      }
@@ -205,15 +205,15 @@ bool refineSequence(std::vector<LocalizationResult>& vec_localizationResult,
         std::size_t maxObs = 0;
         for (const auto landmark : tinyScene.getLandmarks())
         {
-            if (landmark.second.observations.size() > maxObs)
-                maxObs = landmark.second.observations.size();
+            if (landmark.second.getObservations().size() > maxObs)
+                maxObs = landmark.second.getObservations().size();
         }
         namespace bacc = boost::accumulators;
         bacc::accumulator_set<std::size_t, bacc::stats<bacc::tag::mean, bacc::tag::min, bacc::tag::max, bacc::tag::sum>> stats;
         std::vector<std::size_t> hist(maxObs + 1, 0);
         for (const auto landmark : tinyScene.getLandmarks())
         {
-            const std::size_t nobs = landmark.second.observations.size();
+            const std::size_t nobs = landmark.second.getObservations().size();
             assert(nobs < hist.size());
             stats(nobs);
             hist[nobs]++;
@@ -250,7 +250,7 @@ bool refineSequence(std::vector<LocalizationResult>& vec_localizationResult,
 
         for (; iter != endIter;)
         {
-            if (iter->second.observations.size() < minPointVisibility)
+            if (iter->second.getObservations().size() < minPointVisibility)
             {
                 iter = landmarks.erase(iter);
             }

src/aliceVision/multiview/resection/resectionLORansac_test.cpp

@@ -57,7 +57,7 @@ bool refinePoseAsItShouldbe(const Mat& pt3D,
         const size_t idx = vec_inliers[i];
         Landmark landmark;
         landmark.X = pt3D.col(idx);
-        landmark.observations[0] = Observation(pt2D.col(idx), UndefinedIndexT, unknownScale);
+        landmark.getObservations()[0] = Observation(pt2D.col(idx), UndefinedIndexT, unknownScale);
         sfm_data.getLandmarks()[i] = std::move(landmark);
     }
 

src/aliceVision/mvsUtils/MultiViewParams.cpp

@@ -528,7 +528,7 @@ StaticVector<int> MultiViewParams::findNearestCamsFromLandmarks(int rc, int nbNe
 
     for (const auto& landmarkPair : _sfmData.getLandmarks())
     {
-        const auto& observations = landmarkPair.second.observations;
+        const auto& observations = landmarkPair.second.getObservations();
 
         auto viewObsIt = observations.find(viewId);
         if (viewObsIt == observations.end())
@@ -604,7 +604,7 @@ std::vector<int> MultiViewParams::findTileNearestCams(int rc, int nbNearestCams,
 
     for (const auto& landmarkPair : sfmData.getLandmarks())
     {
-        const auto& observations = landmarkPair.second.observations;
+        const auto& observations = landmarkPair.second.getObservations();
 
         auto viewObsIt = observations.find(viewId);
 

src/aliceVision/photometricStereo/normalIntegration.cpp

@@ -543,7 +543,7 @@ void adjustScale(const sfmData::SfMData& sfmData, image::Image<float>& initDepth
         const sfmData::Landmark& currentLandmark = landmarks.at(currentLandmarkIndex);
         knownDepths(i) = pose.depth(currentLandmark.X);
 
-        sfmData::Observation observationInCurrentPicture = currentLandmark.observations.at(viewID);
+        sfmData::Observation observationInCurrentPicture = currentLandmark.getObservations().at(viewID);
 
         int rowInd = observationInCurrentPicture.getY();
         int colInd = observationInCurrentPicture.getX();
@@ -576,7 +576,7 @@ void getZ0FromLandmarks(const sfmData::SfMData& sfmData,
     {
         size_t currentLandmarkIndex = visibleLandmarks.at(i);
         const sfmData::Landmark& currentLandmark = landmarks.at(currentLandmarkIndex);
-        sfmData::Observation observationInCurrentPicture = currentLandmark.observations.at(viewID);
+        sfmData::Observation observationInCurrentPicture = currentLandmark.getObservations().at(viewID);
 
         int rowInd = observationInCurrentPicture.getY();
         int colInd = observationInCurrentPicture.getX();

src/aliceVision/sfm/FrustumFilter.cpp

@@ -169,7 +169,7 @@ void FrustumFilter::init_z_near_z_far_depth(const sfmData::SfMData& sfmData, con
         {
             const sfmData::Landmark& landmark = itL->second;
             const Vec3& X = landmark.X;
-            for (sfmData::Observations::const_iterator iterO = landmark.observations.begin(); iterO != landmark.observations.end(); ++iterO)
+            for (sfmData::Observations::const_iterator iterO = landmark.getObservations().begin(); iterO != landmark.getObservations().end(); ++iterO)
             {
                 const IndexT id_view = iterO->first;
                 const sfmData::Observation& ob = iterO->second;

src/aliceVision/sfm/LocalBundleAdjustmentGraph.cpp

@@ -413,7 +413,7 @@ void LocalBundleAdjustmentGraph::convertDistancesToStates(sfmData::SfMData& sfmD
     for (auto& itLandmark : sfmData.getLandmarks())
     {
         const IndexT landmarkId = itLandmark.first;
-        const sfmData::Observations& observations = itLandmark.second.observations;
+        const sfmData::Observations& observations = itLandmark.second.getObservations();
 
         assert(observations.size() >= 2);
 
@@ -475,7 +475,7 @@ std::vector<Pair> LocalBundleAdjustmentGraph::getNewEdges(const sfmData::SfMData
         // retrieve the common track Ids
         for (IndexT landmarkId : newViewLandmarks)
         {
-            for (const auto& observations : sfmData.getLandmarks().at(landmarkId).observations)
+            for (const auto& observations : sfmData.getLandmarks().at(landmarkId).getObservations())
             {
                 if (observations.first == viewId)
                     continue;  // do not compare an observation with itself

src/aliceVision/sfm/bundle/BundleAdjustmentCeres.cpp

@@ -806,7 +806,7 @@ void BundleAdjustmentCeres::addLandmarksToProblem(const sfmData::SfMData& sfmDat
         _allParametersBlocks.push_back(landmarkBlockPtr);
 
         // iterate over 2D observation associated to the 3D landmark
-        for (const auto& observationPair : landmark.observations)
+        for (const auto& observationPair : landmark.getObservations())
         {
             const sfmData::View& view = sfmData.getView(observationPair.first);
             const sfmData::Observation& observation = observationPair.second;

src/aliceVision/sfm/bundle/BundleAdjustmentSymbolicCeres.cpp

@@ -491,7 +491,7 @@ void BundleAdjustmentSymbolicCeres::addLandmarksToProblem(const sfmData::SfMData
         _allParametersBlocks.push_back(landmarkBlockPtr);
 
         // iterate over 2D observation associated to the 3D landmark
-        for (const auto& observationPair : landmark.observations)
+        for (const auto& observationPair : landmark.getObservations())
         {
             const sfmData::View& view = sfmData.getView(observationPair.first);
             const sfmData::Observation& observation = observationPair.second;

src/aliceVision/sfm/bundle/bundleAdjustment_test.cpp

@@ -154,12 +154,12 @@ BOOST_AUTO_TEST_CASE(LOCAL_BUNDLE_ADJUSTMENT_EffectiveMinimization_Pinhole_Camer
     //          |     |     =>     /  \ /  \ /  \
   //          v1 - v2           v0   v1   v2   v3
     // removing adequate observations:
-    sfmData.getLandmarks().at(0).observations.erase(2);
-    sfmData.getLandmarks().at(0).observations.erase(3);
-    sfmData.getLandmarks().at(1).observations.erase(0);
-    sfmData.getLandmarks().at(1).observations.erase(3);
-    sfmData.getLandmarks().at(2).observations.erase(0);
-    sfmData.getLandmarks().at(2).observations.erase(1);
+    sfmData.getLandmarks().at(0).getObservations().erase(2);
+    sfmData.getLandmarks().at(0).getObservations().erase(3);
+    sfmData.getLandmarks().at(1).getObservations().erase(0);
+    sfmData.getLandmarks().at(1).getObservations().erase(3);
+    sfmData.getLandmarks().at(2).getObservations().erase(0);
+    sfmData.getLandmarks().at(2).getObservations().erase(1);
 
     // lock common intrinsic
     // if it's not locked, all views will have a distance of 1 as all views share a common intrinsic.
@@ -248,7 +248,7 @@ double RMSE(const SfMData& sfm_data)
     std::vector<double> vec;
     for (Landmarks::const_iterator iterTracks = sfm_data.getLandmarks().begin(); iterTracks != sfm_data.getLandmarks().end(); ++iterTracks)
     {
-        const Observations& observations = iterTracks->second.observations;
+        const Observations& observations = iterTracks->second.getObservations();
         for (Observations::const_iterator itObs = observations.begin(); itObs != observations.end(); ++itObs)
         {
             const View* view = sfm_data.getViews().find(itObs->first)->second.get();
@@ -315,7 +315,7 @@ SfMData getInputScene(const NViewDataSet& d, const NViewDatasetConfigurator& con
             pt(0) += rand() / RAND_MAX - .5;
             pt(1) += rand() / RAND_MAX - .5;
 
-            landmark.observations[j] = Observation(pt, i, unknownScale);
+            landmark.getObservations()[j] = Observation(pt, i, unknownScale);
         }
         sfm_data.getLandmarks()[i] = landmark;
     }

src/aliceVision/sfm/generateReport.cpp

@@ -26,7 +26,7 @@ bool generateSfMReport(const sfmData::SfMData& sfmData, const std::string& htmlF
     HashMap<IndexT, std::vector<double>> residuals_per_view;
     for (sfmData::Landmarks::const_iterator iterTracks = sfmData.getLandmarks().begin(); iterTracks != sfmData.getLandmarks().end(); ++iterTracks)
     {
-        const sfmData::Observations& observations = iterTracks->second.observations;
+        const sfmData::Observations& observations = iterTracks->second.getObservations();
         for (sfmData::Observations::const_iterator itObs = observations.begin(); itObs != observations.end(); ++itObs)
         {
             const sfmData::View* view = sfmData.getViews().at(itObs->first).get();