Changeset 1056 for cpp/frams/model

Timestamp:

12/28/20 02:06:52 (3 years ago)

Author:

Maciej Komosinski

Message:

Cosmetic

Location:

cpp/frams/model/geometry

Files:

• geometryutils.cpp (modified) (3 diffs)
• geometryutils.h (modified) (1 diff)
• part_distance_estimator.h (modified) (2 diffs)

cpp/frams/model/geometry/geometryutils.cpp

@@ -369 +369 @@
 }
 
-double GeometryUtils::calculateSolidVolume(Part * part)
+double GeometryUtils::calculateSolidVolume(const Part * part)
 {
         double radiiProduct = part->scale.x * part->scale.y * part->scale.z;
@@ -386 +386 @@
 }
 
-bool GeometryUtils::isSolidPartScaleValid(const Part::Shape &partShape, const Pt3D &scale)
+bool GeometryUtils::isSolidPartScaleValid(const Part::Shape &partShape, const Pt3D &scale, bool ensureCircleSection)
 {
         Part *tmpPart = new Part(partShape);
@@ -402 +402 @@
                 return false;
 
-        if (partShape == Part::Shape::SHAPE_ELLIPSOID && scale.maxComponentValue() != scale.minComponentValue()) // When any radius has a different value than the others
-                return false;
-        if (partShape == Part::Shape::SHAPE_CYLINDER && scale.y != scale.z) // If base radii have different values
-                return false;
+        if (ensureCircleSection)
+        {
+                if (partShape == Part::Shape::SHAPE_ELLIPSOID && scale.maxComponentValue() != scale.minComponentValue()) // Any radius has a different value than the other ones?
+                        return false;
+                if (partShape == Part::Shape::SHAPE_CYLINDER && scale.y != scale.z) // Base radii have different values?
+                        return false;
+        }
         return true;
 }

cpp/frams/model/geometry/geometryutils.h

@@ -178 +178 @@
         double ellipsePerimeter(const double a, const double b);
 
-        double calculateSolidVolume(Part *part);
-        bool isSolidPartScaleValid(const Part::Shape &partShape, const Pt3D &scale);
+        double calculateSolidVolume(const Part *part);
+        bool isSolidPartScaleValid(const Part::Shape &partShape, const Pt3D &scale, bool ensureCircleSection);
 
     /**

cpp/frams/model/geometry/part_distance_estimator.h

@@ -7 +7 @@
 
 #include "frams/model/geometry/meshbuilder.h"
+#include <cmath>
 
 class PartDistanceEstimator
@@ -54 +55 @@
         static double calculateDistance(Part tmpPart1, Part tmpPart2, double distanceTolerance, double relativeDensity)
         {
-                /// tmpPart1 and tmpPart2 are copied for purpose and should not be passed as reference
-                /// This function can change some of the properties of those parts
+                /// tmpPart1 and tmpPart2 are copied for purpose and should not be passed as reference.
+                /// This function can change some of the properties of those parts.
                 /// tmpPart1 will be approximated by surface points.
-                /// The collision between the parts is detected when any of those points is inside tmpPart2
-                /// If tmpPart1 and tmpPart2 are swapped, the calculated distance may slightly differ
+                /// The collision between the parts is detected when any of these points is inside tmpPart2.
+                /// If tmpPart1 and tmpPart2 are swapped, the calculated distance may slightly differ.
                 Pt3D directionVersor = tmpPart1.p - tmpPart2.p;
                 directionVersor.normalize();