Merge pull request #1548 from dcoeurjo/FixReg

Bugfix in Digital Surface Regularization
DGtal-team · Feb 3, 2021 · f90f644 · f90f644
2 parents 87d3b02 + cfbe496
commit f90f644
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/ChangeLog.md b/ChangeLog.md
@@ -51,7 +51,9 @@
  (Jacques-Olivier Lachaud,[#1538](https:/DGtal-team/DGtal/pull/1538))
  - Fix BoundedLatticePolytope::init when using half-spaces initialization
  (Jacques-Olivier Lachaud,[#1531](https:/DGtal-team/DGtal/pull/1531))
-
+ - Fix an issue in DigitalSurfaceRegularization about bad buffer init
+ (David Coeurjolly, [#1548](https:/DGtal-team/DGtal/pull/1548))
+
 - *Shapes package*
  - Fix the use of uninitialized variable in NGon2D.
  (Daniel Antunes,[#1540](https:/DGtal-team/DGtal/issues/1540))

diff --git a/src/DGtal/geometry/surfaces/DigitalSurfaceRegularization.ih b/src/DGtal/geometry/surfaces/DigitalSurfaceRegularization.ih
@@ -107,10 +107,10 @@ void DGtal::DigitalSurfaceRegularization<T>::cacheInit()
  std::copy(myOriginalPositions.begin(), myOriginalPositions.end(), myRegularizedPositions.begin());
 
  //Allocating Gradient vector
- myGradient.resize(myOriginalPositions.size());
  myGradient.clear();
+ myGradientAlign.clear();
+ myGradient.resize(myOriginalPositions.size());
  myGradientAlign.resize(myOriginalPositions.size());
- myGradient.clear();
 
  /////
  ///Cacheing some topological information