Image segment profile: fix point coordinates inversion

Fix #88

Author: PierreRaybaut

CHANGELOG.md

@@ -15,6 +15,7 @@ See DataLab [roadmap page](https://datalab-platform.com/en/contributing/roadmap.
   * Thanks again to [@rolandmas](https://github.com/rolandmas) for reporting the issue in the context of the Debian packaging
 * Fixed [Issue #86](https://github.com/DataLab-Platform/DataLab/issues/86) - Average of N integer images overflows data type
 * Fixed [Issue #87](https://github.com/DataLab-Platform/DataLab/issues/87) - Image average profile extraction: `AttributeError` when trying to edit profile parameters
+* Fixed [Issue #88](https://github.com/DataLab-Platform/DataLab/issues/88) - Image segment profile: point coordinates inversion
 
 ## DataLab Version 0.16.2 ##
 

cdl/core/gui/profiledialog.py

@@ -203,13 +203,19 @@ def shape_to_param(self) -> None:
         elif isinstance(self.shape, AnnotatedSegment):
             assert isinstance(p, cdl.param.SegmentProfileParam)
             x1, y1, x2, y2 = self.shape.get_rect()
-            p.row1, p.row2 = sorted([int(np.round(y1)), int(np.round(y2))])
-            p.col1, p.col2 = sorted([int(np.round(x1)), int(np.round(x2))])
+            p.row1, p.row2 = int(np.round(y1)), int(np.round(y2))
+            p.col1, p.col2 = int(np.round(x1)), int(np.round(x2))
+            if p.col1 > p.col2:
+                p.col1, p.col2 = p.col2, p.col1
+                p.row1, p.row2 = p.row2, p.row1
         else:
             assert isinstance(p, cdl.param.AverageProfileParam)
             x1, y1, x2, y2 = self.shape.get_rect()
-            p.row1, p.row2 = sorted([int(np.round(y1)), int(np.round(y2))])
-            p.col1, p.col2 = sorted([int(np.round(x1)), int(np.round(x2))])
+            p.row1, p.row2 = int(np.round(y1)), int(np.round(y2))
+            p.col1, p.col2 = int(np.round(x1)), int(np.round(x2))
+            if p.col1 > p.col2:
+                p.col1, p.col2 = p.col2, p.col1
+                p.row1, p.row2 = p.row2, p.row1
 
     def param_to_shape(self) -> None:
         """Param to shape"""