Added point type to geometry classes parameters.

Fixes #368

Author: maisonobe

hipparchus-geometry/src/changes/changes.xml

@@ -49,6 +49,12 @@ If the output is not quite correct, check for invisible trailing spaces!
     <title>Hipparchus Geometry Release Notes</title>
   </properties>
   <release version="4.0" date="TBD" description="TBD">
+    <action dev="luc" type="fix" issue="issues/368">
+      Added point type to geometry classes parameters
+    </action>
+    <action dev="luc" type="update" issue="issues/314">
+      Improved robustness of BSP tree operations
+    </action>
     <action dev="vincent" type="update" issue="issues/285">
       Migrated tests from JUnit 4 to JUnit 5
     </action>

hipparchus-geometry/src/site/design/oneD.puml

@@ -36,7 +36,7 @@
       space with (d-1)-dimensional space
     end note
 
-    interface "Vector<S extends Space>" as Vector_S_ {
+    interface "Vector<S extends Space, P extends Point<S>>" as Vector_S_ {
       +Space getSpace()
       +Vector getZero()
       +double getNorm()
@@ -54,9 +54,9 @@
     Space <-- Vector_S_
 
     package partitioning #DDEBD8 {
-      interface "Region<S extends Space>" as Region_S_
-      interface "Hyperplane<S extends Space>" as Hyperplane_S_
-      interface "SubHyperplane<S extends Space>" as SubHyperplane_S_
+      interface "Region<S extends Space, P extends Point<S>>" as Region_S_
+      interface "Hyperplane<S extends Space, P extends Point<S>>" as Hyperplane_S_
+      interface "SubHyperplane<S extends Space, P extends Point<S>>" as SubHyperplane_S_
     }
 
     package euclidean #DDEBD8 {

hipparchus-geometry/src/site/design/partitioning.puml

@@ -28,22 +28,22 @@
 
     package partitioning #DDEBD8 {
 
-      abstract "AbstractSubHyperplane<S extends Space, T extends Space>" as AbstractSubHyperplane_S_T_
+      abstract "AbstractSubHyperplane<S extends Space, P extends Point<S>, T extends Space, Q extends Point<T>>" as AbstractSubHyperplane_S_T_
       note left
         an abstract sub-hyperplane contains
          - an hyperplane defined in space S
          - a region defined in space T
         T being a sub-space of S
       end note
 
-      interface "Hyperplane<S extends Space>" as Hyperplane_S_ {
+      interface "Hyperplane<S extends Space, P extends Point<S>>" as Hyperplane_S_ {
         +double getOffset(Vector point)
         +boolean sameOrientationAs(Hyperplane other)
         +SubHyperplane wholeHyperplane()
         +Region wholeSpace()
       }
 
-      interface "SubHyperplane<S extends Space>" as SubHyperplane_S_ {
+      interface "SubHyperplane<S extends Space, P extends Point<S>>" as SubHyperplane_S_ {
         +Hyperplane getHyperplane()
         +boolean isEmpty()
         +double getSize()
@@ -52,7 +52,7 @@
         +SubHyperplane reunite(SubHyperplane other)
       }
 
-      class "BSPTree<S extends Space>" as BSPTree_S_ {
+      class "BSPTree<S extends Space, P extends Point<S>>" as BSPTree_S_ {
         +boolean insertCut(Hyperplane hyperplane)
         +void setAttribute(Object attribute)
         +Object getAttribute()
@@ -61,19 +61,19 @@
         +BSPTree split(SubHyperplane sub)
       }
 
-      interface "BSPTreeVisitor<S extends Space>" as BSPTreeVisitor_S_ {
+      interface "BSPTreeVisitor<S extends Space, P extends Point<S>>" as BSPTreeVisitor_S_ {
         +Order visitOrder(BSPTree node)
         +void visitInternalNode(BSPTree node)
         +void visitLeafNode(BSPTree node)
       }
 
-      interface "Region<S extends Space>" as Region_S_ {
+      interface "Region<S extends Space, P extends Point<S>>" as Region_S_ {
         +boolean isEmpty()
         +boolean contains(Region region)
-        +Location checkPoint(Vector point)
+        +Location checkPoint(P point)
         +double getBoundarySize()
         +double getSize()
-        +Vector getBarycenter()
+        +P getBarycenter()
         +Side side(Hyperplane hyperplane)
         +SubHyperplane intersection(SubHyperplane sub)
       }

hipparchus-geometry/src/site/design/threeD.puml

@@ -36,7 +36,7 @@
       space with (d-1)-dimensional space
     end note
 
-    interface "Vector<S extends Space>" as Vector_S_ {
+    interface "Vector<S extends Space, P extends Point<S>>" as Vector_S_ {
       +Space getSpace()
       +Vector getZero()
       +double getNorm()
@@ -54,9 +54,9 @@
     Space <-- Vector_S_
 
     package partitioning #DDEBD8 {
-      interface "Region<S extends Space>" as Region_S_
-      interface "Hyperplane<S extends Space>" as Hyperplane_S_
-      interface "SubHyperplane<S extends Space>" as SubHyperplane_S_
+      interface "Region<S extends Space, P extends Point<S>>" as Region_S_
+      interface "Hyperplane<S extends Space, P extends Point<S>>" as Hyperplane_S_
+      interface "SubHyperplane<S extends Space, P extends Point<S>>" as SubHyperplane_S_
     }
 
     package euclidean #DDEBD8 {

hipparchus-geometry/src/site/design/twoD.puml

@@ -36,7 +36,7 @@
       space with (d-1)-dimensional space
     end note
 
-    interface "Vector<S extends Space>" as Vector_S_ {
+    interface "Vector<S extends Space, P extends Point<S>>" as Vector_S_ {
       +Space getSpace()
       +Vector getZero()
       +double getNorm()
@@ -54,9 +54,9 @@
     Space <-- Vector_S_
 
     package partitioning #DDEBD8 {
-      interface "Region<S extends Space>" as Region_S_
-      interface "Hyperplane<S extends Space>" as Hyperplane_S_
-      interface "SubHyperplane<S extends Space>" as SubHyperplane_S_
+      interface "Region<S extends Space, P extends Point<S>>" as Region_S_
+      interface "Hyperplane<S extends Space, P extends Point<S>>" as Hyperplane_S_
+      interface "SubHyperplane<S extends Space, P extends Point<S>>" as SubHyperplane_S_
     }
 
     package euclidean #DDEBD8 {

src/changes/changes.xml

@@ -50,6 +50,12 @@ If the output is not quite correct, check for invisible trailing spaces!
   </properties>
   <body>
     <release version="4.0" date="TBD" description="TBD">
+      <action dev="luc" type="fix" issue="issues/368">
+        Added point type to geometry classes parameters
+      </action>
+      <action dev="luc" type="update" issue="issues/314">
+        Improved robustness of BSP tree operations
+      </action>
       <action dev="serrof" type="update" issue="issues/361">
         Rename DEFAULT_MAXCHECK as DEFAULT_MAX_CHECK.
       </action>