Merge pull request #14 from ahewer/master

fix: landmark is no longer placed between voxels
m2ci-msp · May 22, 2018 · 56aafdb · 56aafdb
2 parents c323bbc + f62c727
commit 56aafdb
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 8 deletions.
diff --git a/landmark-tool/src/include/action/AddLandmark.h b/landmark-tool/src/include/action/AddLandmark.h
@@ -42,10 +42,10 @@ class AddLandmark : public Action {
 
       auto point = EventPosition::get_instance()->get(); 
 
-      // convert point to the center of the pixel containing it
-      point.set_x( (int) point.get_x() + 0.5);
-      point.set_y( (int) point.get_y() + 0.5);
-      point.set_z( (int) point.get_z() + 0.5);
+      // convert point to the voxel containing it
+      point.set_x( (int) point.get_x() );
+      point.set_y( (int) point.get_y() );
+      point.set_z( (int) point.get_z() );
 
       // create landmark and add it to the landmark pool
       auto mark = LandmarkPool::get_instance()->create_landmark(point); 

diff --git a/landmark-tool/src/include/action/MoveLandmark.h b/landmark-tool/src/include/action/MoveLandmark.h
@@ -27,10 +27,10 @@ class MoveLandmark: public Action {
 
       auto point = EventPosition::get_instance()->get(); 
 
-      // convert point to the center of the pixel containing it
-      point.set_x( (int) point.get_x() + 0.5);
-      point.set_y( (int) point.get_y() + 0.5);
-      point.set_z( (int) point.get_z() + 0.5);
+      // convert point to voxel containing it
+      point.set_x( (int) point.get_x() );
+      point.set_y( (int) point.get_y() );
+      point.set_z( (int) point.get_z() );
 
       auto mark = LandmarkPool::get_instance()->get_selected_landmark();