3 bug vortexplottinggauss to ellipse is not exported correctly (#6)

* changed cmakelists and ellipse class

* package is successfully imported by other packages

* Committing clang-format changes

* fixed scaling factor in ellipse from gauss constructor

* Committing clang-format changes

* updated clang formatter workflow file

---------

Co-authored-by: Clang Robot <[email protected]>

.github/workflows/clang-formatter.yml

@@ -1,19 +1,19 @@
 name: Run clang-format Linter
 
 on: 
-  # push:
-    # branches: [ main ]
+  push:
+    branches: [ main ]
   workflow_dispatch:
 
-  pull_request:
-    types: [opened, reopened]
-    branches: [ main ]
+  # pull_request:
+  #   types: [opened, reopened]
+  #   branches: [ main ]
 jobs:
   build:
     runs-on: ubuntu-latest
 
     steps:
-    - uses: actions/checkout@v3
+    - uses: actions/checkout@v4
     - uses: DoozyX/[email protected]
       with:
         source: '.'

vortex-filtering/include/vortex_filtering/utils/ellipse.hpp

@@ -22,16 +22,16 @@ namespace utils {
 class Ellipse {
 public:
   /** Construct a new Ellipse object
-   * @param center
-   * @param a
-   * @param b
-   * @param angle
+   * @param center center of the ellipse
+   * @param a half the length of the major axis (radius of the circumscribed circle)
+   * @param b half the length of the minor axis (radius of the inscribed circle)
+   * @param angle angle in radians
    */
   Ellipse(const Eigen::Vector2d &center, double a, double b, double angle);
 
   /** Construct a new Ellipse object from a Gaussian
-   * @param gauss
-   * @param scale_factor
+   * @param gauss 2D Gaussian distribution
+   * @param scale_factor scale factor for the ellipse
    */
   Ellipse(const vortex::prob::Gauss2d &gauss, double scale_factor = 1.0);
 
@@ -50,12 +50,12 @@ class Ellipse {
    */
   double y() const;
 
-  /** Get the a parameter of the ellipse
+  /** Get the a parameter of the ellipse (half the length of the major axis)
    * @return double
    */
   double a() const;
 
-  /** Get the b parameter of the ellipse
+  /** Get the b parameter of the ellipse (half the length of the minor axis)
    * @return double
    */
   double b() const;

vortex-filtering/src/utils/ellipse.cpp

@@ -11,8 +11,14 @@ Ellipse::Ellipse(const vortex::prob::Gauss2d &gauss, double scale_factor)
   Eigen::Vector2d eigenValues  = eigenSolver.eigenvalues();
   Eigen::Matrix2d eigenVectors = eigenSolver.eigenvectors();
 
-  a_      = scale_factor * sqrt(eigenValues(1));
-  b_      = scale_factor * sqrt(eigenValues(0));
+  // Sort eigenvalues in descending order
+  if (eigenValues(0) > eigenValues(1)) {
+    std::swap(eigenValues(0), eigenValues(1));
+    eigenVectors.col(0).swap(eigenVectors.col(1));
+  }
+
+  a_      = sqrt(scale_factor * eigenValues(1));
+  b_      = sqrt(scale_factor * eigenValues(0));
   angle_  = atan2(eigenVectors(1, 1), eigenVectors(0, 1));
   center_ = gauss.mean();
 }