Skip to content

Commit

Permalink
Remove the requirement for 3 touch points
Browse files Browse the repository at this point in the history
  • Loading branch information
mpowelson committed Jan 23, 2020
1 parent bad06a9 commit d4f7c2a
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion opp_gui/src/widgets/tool_path_planner_widget.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,8 @@
#include "ui_tool_path_planner.h"

const static std::string MESH_MARKER_TOPIC = "mesh_marker";
const static int MIN_TOUCH_POINTS = 0;
const static int MIN_VERIFICATION_POINTS = 0;

namespace opp_gui
{
Expand Down Expand Up @@ -303,7 +305,7 @@ void ToolPathPlannerWidget::saveModel()
using TouchPointMap = std::map<std::string, opp_msgs::TouchPoint>;
TouchPointMap touch_points = touch_point_editor_->getPoints();
TouchPointMap verification_points = verification_point_editor_->getPoints();
if (touch_points.size() < 3 || verification_points.size() < 3)
if (touch_points.size() < MIN_TOUCH_POINTS || verification_points.size() < MIN_VERIFICATION_POINTS)
{
QMessageBox::warning(
this, "Invalid Model Definition", "Ensure at least 3 touch points and 3 verification points have been defined");
Expand Down

0 comments on commit d4f7c2a

Please sign in to comment.