Fix comments from pull request

Brendan Klare
1 parent a0bb95c7
Showing 4 changed files with 13 additions and 11 deletions
openbr/core/eigenutils.cpp
openbr/core/eigenutils.h
openbr/core/opencvutils.cpp
openbr/core/opencvutils.h
@@ -137,3 +137,11 @@ Eigen::MatrixXf matrixToVector(const Eigen::MatrixXf matrix) {
     return vector;
 }
+Eigen::MatrixXf toEigen(const Mat m) {
+    if (m.type() != CV_32F)
+        qFatal("Mat to Eigen Converstation only supports CV_32F");
+
+    Eigen::MatrixXf data(m.rows, m.cols);
+    return Eigen::Map<const Eigen::MatrixXf>(m.ptr<float>(), m.rows, m.cols);
+}
+
@@ -21,6 +21,8 @@
 #include <Eigen/Core>
 #include <assert.h>
+#include <opencv2/core/core.hpp>
+
 void writeEigen(Eigen::MatrixXf X, QString filename);
 void writeEigen(Eigen::MatrixXd X, QString filename);
 void writeEigen(Eigen::VectorXd X, QString filename);
@@ -40,6 +42,9 @@ Eigen::MatrixXf removeRowCol(const Eigen::MatrixXf X, int row, int col);
 Eigen::MatrixXf pointsToMatrix(const QList<QPointF> points, bool isAffine=false);
 QList<QPointF> matrixToPoints(const Eigen::MatrixXf P);
+//Convert cv::Mat to Eigen
+Eigen::MatrixXf toEigen(const cv::Mat m);
+
 template<typename _Scalar, int _Rows, int _Cols, int _Options, int _MaxRows, int _MaxCols>
 inline QDataStream &operator<<(QDataStream &stream, const Eigen::Matrix< _Scalar, _Rows, _Cols, _Options, _MaxRows, _MaxCols > &mat)
 {
@@ -336,14 +336,6 @@ bool OpenCVUtils::overlaps(const QList&lt;Rect&gt; &amp;posRects, const Rect &amp;negRect, dou
     return false;
 }
-Eigen::MatrixXf OpenCVUtils::toEigen(Mat m) {
-    if (m.type() != CV_32F)
-        qFatal("Mat to Eigen Converstation only supports CV_32F");
-
-    Eigen::MatrixXf data(m.rows, m.cols);
-    return Eigen::Map<const Eigen::MatrixXf>(m.ptr<float>(), m.rows, m.cols);
-}
-
 QDataStream &operator<<(QDataStream &stream, const Mat &m)
 {
     // Write header
@@ -24,8 +24,6 @@
 #include <opencv2/core/core.hpp>
 #include <assert.h>
-#include <Eigen/Core>
-
 namespace OpenCVUtils
 {
     // Test/write/display image
@@ -93,7 +91,6 @@ namespace OpenCVUtils
     bool overlaps(const QList<cv::Rect> &posRects, const cv::Rect &negRect, double overlap);
     float overlap(const cv::Rect &rect1, const cv::Rect &rect2);
     float overlap(const QRectF &rect1, const QRectF &rect2);
-    Eigen::MatrixXf toEigen(cv::Mat m);
     int getFourcc();
 }