From 12baf54d17347c868dc026c22ba8f3b8761c327a Mon Sep 17 00:00:00 2001 From: Varun Agrawal Date: Sat, 17 May 2025 00:01:02 -0400 Subject: [PATCH] rename kRandomNumberGenerator in local files to kPRNG to prevent collision --- gtsam/sfm/ShonanAveraging.cpp | 6 +++--- gtsam/sfm/TranslationRecovery.cpp | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/gtsam/sfm/ShonanAveraging.cpp b/gtsam/sfm/ShonanAveraging.cpp index 7d68c7d29..b268fda6f 100644 --- a/gtsam/sfm/ShonanAveraging.cpp +++ b/gtsam/sfm/ShonanAveraging.cpp @@ -42,7 +42,7 @@ namespace gtsam { // In Wrappers we have no access to this so have a default ready -static std::mt19937 kRandomNumberGenerator(42); +static std::mt19937 kPRNG(42); using Sparse = Eigen::SparseMatrix; @@ -869,7 +869,7 @@ Values ShonanAveraging::initializeRandomly(std::mt19937 &rng) const { /* ************************************************************************* */ template Values ShonanAveraging::initializeRandomly() const { - return initializeRandomly(kRandomNumberGenerator); + return initializeRandomly(kPRNG); } /* ************************************************************************* */ @@ -883,7 +883,7 @@ Values ShonanAveraging::initializeRandomlyAt(size_t p, /* ************************************************************************* */ template Values ShonanAveraging::initializeRandomlyAt(size_t p) const { - return initializeRandomlyAt(p, kRandomNumberGenerator); + return initializeRandomlyAt(p, kPRNG); } /* ************************************************************************* */ diff --git a/gtsam/sfm/TranslationRecovery.cpp b/gtsam/sfm/TranslationRecovery.cpp index 02c78133e..828aa6880 100644 --- a/gtsam/sfm/TranslationRecovery.cpp +++ b/gtsam/sfm/TranslationRecovery.cpp @@ -39,7 +39,7 @@ using namespace gtsam; using namespace std; // In Wrappers we have no access to this so have a default ready. -static std::mt19937 kRandomNumberGenerator(42); +static std::mt19937 kPRNG(42); // Some relative translations may be zero. We treat nodes that have a zero // relativeTranslation as a single node. @@ -185,7 +185,7 @@ Values TranslationRecovery::initializeRandomly( const std::vector> &betweenTranslations, const Values &initialValues) const { return initializeRandomly(relativeTranslations, betweenTranslations, - &kRandomNumberGenerator, initialValues); + &kPRNG, initialValues); } Values TranslationRecovery::run(