From 0d2dcc42f58a1a5ac1a22544de95fb5f931e71f6 Mon Sep 17 00:00:00 2001
From: Peter Albrecht <albrecpe@gmail.com>
Date: Fri, 6 Sep 2024 16:11:15 +0200
Subject: [PATCH] Remove even more prints
---
.../AronTreeWidgetController.cpp | 3 ---
.../visitors/AronTreeWidgetConverter.cpp | 20 ++++---------------
.../visitors/AronTreeWidgetSetter.cpp | 18 -----------------
3 files changed, 4 insertions(+), 37 deletions(-)
diff --git a/source/RobotAPI/libraries/skills_gui/aron_tree_widget/AronTreeWidgetController.cpp b/source/RobotAPI/libraries/skills_gui/aron_tree_widget/AronTreeWidgetController.cpp
index 97218df98..8c24f906f 100644
--- a/source/RobotAPI/libraries/skills_gui/aron_tree_widget/AronTreeWidgetController.cpp
+++ b/source/RobotAPI/libraries/skills_gui/aron_tree_widget/AronTreeWidgetController.cpp
@@ -145,18 +145,15 @@ namespace armarx::skills::gui
aronElem->onUserChange(column);
}
- ARMARX_INFO << "After onUserChange";
// start conversion for entire tree -- this also sets the highlighting
if (parent->childCount() == 1)
{
auto* aronTreeRoot = AronTreeWidgetItem::DynamicCast(parent->child(0));
if (aronTreeRoot)
{
- ARMARX_INFO << "Looking at aronTreeRoot";
aronTreeRoot->resetError();
AronTreeWidgetConverterVisitor v(parent, 0);
aron::type::visit(v, type);
- ARMARX_INFO << "After visiting";
aronTreeRoot->setValueErrorState(v.hasDirectError(), v.onlyChildFailedConversion());
}
}
diff --git a/source/RobotAPI/libraries/skills_gui/aron_tree_widget/visitors/AronTreeWidgetConverter.cpp b/source/RobotAPI/libraries/skills_gui/aron_tree_widget/visitors/AronTreeWidgetConverter.cpp
index fac283357..b4c2802ec 100644
--- a/source/RobotAPI/libraries/skills_gui/aron_tree_widget/visitors/AronTreeWidgetConverter.cpp
+++ b/source/RobotAPI/libraries/skills_gui/aron_tree_widget/visitors/AronTreeWidgetConverter.cpp
@@ -289,8 +289,7 @@ namespace armarx::skills::gui
break;
};
- ARMARX_INFO << "Matrix: 1";
-
+
// UGLY HACK: FIX ME!!!
switch (i->getElementType())
{
@@ -323,8 +322,7 @@ namespace armarx::skills::gui
break;
};
- ARMARX_INFO << "Matrix: 2";
-
+
createdMatrix->setShape({i->getRows(), i->getCols(), dataSize});
int totalByteSize = i->getRows() * i->getCols() * dataSize;
createdAron = createdMatrix;
@@ -340,8 +338,7 @@ namespace armarx::skills::gui
}
}
- ARMARX_INFO << "Matrix: 3";
-
+
auto* rootWidget = el->treeWidget();
ARMARX_CHECK(rootWidget);
auto* widget = rootWidget->itemWidget(el, 1);
@@ -355,8 +352,7 @@ namespace armarx::skills::gui
return;
}
- ARMARX_INFO << "Matrix: 4";
-
+
// write to aron data
std::vector<unsigned char> elems;
elems.reserve(totalByteSize);
@@ -675,11 +671,6 @@ namespace armarx::skills::gui
return;
}
- ARMARX_INFO << "el = " << el;
- ARMARX_INFO << "index = " << index;
-
- ARMARX_INFO << "String: 1";
-
if (i->getMaybe() != armarx::aron::type::Maybe::NONE)
{
// its a maybetype. We have to check the state
@@ -690,10 +681,7 @@ namespace armarx::skills::gui
}
}
- ARMARX_INFO << "String: 2";
-
std::string str = el->text(1).toStdString();
- ARMARX_INFO << "createdAronString = " << createdAronString;
createdAronString->setValue(str);
}
diff --git a/source/RobotAPI/libraries/skills_gui/aron_tree_widget/visitors/AronTreeWidgetSetter.cpp b/source/RobotAPI/libraries/skills_gui/aron_tree_widget/visitors/AronTreeWidgetSetter.cpp
index fec0fdf55..373086b4f 100644
--- a/source/RobotAPI/libraries/skills_gui/aron_tree_widget/visitors/AronTreeWidgetSetter.cpp
+++ b/source/RobotAPI/libraries/skills_gui/aron_tree_widget/visitors/AronTreeWidgetSetter.cpp
@@ -39,8 +39,6 @@ usString(T number, size_t precision = 3)
std::stringstream ss;
const char* locale = "C";
ss.imbue(std::locale(locale));
-
- ARMARX_INFO << "Building StringStream";
ss << std::fixed << std::setprecision(precision) << number;
return ss.str();
@@ -139,15 +137,10 @@ namespace armarx::skills::gui
if (checkTreeWidgetItemForSimilarName(i->getPath().getLastElement()))
{
- ARMARX_INFO << "Marker 0";
AronTreeWidgetItem* el = AronTreeWidgetItem::DynamicCast(parentItem->child(index));
- ARMARX_INFO << "el: " << el;
auto* aronTreeWidget = AronTreeWidgetItem::DynamicCastAndCheck(el);
- ARMARX_INFO << "aronTreeWidget: " << aronTreeWidget;
adjustNumberOfChildren(aronTreeWidget, i->childrenSize());
- ARMARX_INFO << "Marker 1";
-
unsigned int x = 0;
for (const auto& value : i->getElements())
{
@@ -159,7 +152,6 @@ namespace armarx::skills::gui
++x;
}
- ARMARX_INFO << "Marker 2";
// This displays the number of children also when the list is collapsed
QString numElemsText = misc::generateNumElementsText(i->getElements().size());
aronTreeWidget->setText(1, numElemsText);
@@ -383,24 +375,14 @@ namespace armarx::skills::gui
void
AronTreeWidgetSetterVisitor::visitAronVariant(const aron::data::FloatPtr& i)
{
-
- ARMARX_INFO << "path: " << i->getPath().toString();
- ARMARX_INFO << "value: " << i->getValue();
- ARMARX_INFO << "test: " << usString<float>(i->getValue());
-
if (checkTreeWidgetItemForSimilarName(i->getPath().getLastElement()))
{
AronTreeWidgetItem* el = AronTreeWidgetItem::DynamicCast(parentItem->child(index));
- ARMARX_INFO << "el = " << el;
std::string repr = usString<float>(i->getValue());
- ARMARX_INFO << "repr = " << repr;
QString qstr = QString::fromStdString(repr);
- ARMARX_INFO << "qstr = " << qstr;
el->setText(1, qstr);
- ARMARX_INFO << "After SetText";
if (el->aronType && el->aronType->getMaybe() != armarx::aron::type::Maybe::NONE)
{
- ARMARX_INFO << "What?";
el->setCheckState(1, Qt::CheckState::Checked);
}
}
--
GitLab