From a1ea89242f33554b32bd09608a90216ad510dc69 Mon Sep 17 00:00:00 2001
From: Dominik Prossel <dominik.prossel@kit.edu>
Date: Thu, 29 Jun 2023 16:59:41 +0200
Subject: [PATCH] fix querying object descriptions with instance name
---
.../armem_objects/client/articulated_object/Reader.cpp | 10 ++++------
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/source/RobotAPI/libraries/armem_objects/client/articulated_object/Reader.cpp b/source/RobotAPI/libraries/armem_objects/client/articulated_object/Reader.cpp
index 9a4417080..b23673cb0 100644
--- a/source/RobotAPI/libraries/armem_objects/client/articulated_object/Reader.cpp
+++ b/source/RobotAPI/libraries/armem_objects/client/articulated_object/Reader.cpp
@@ -120,11 +120,11 @@ namespace armarx::armem::articulated_object
const std::string className = splits.at(0) + "/" + splits.at(1); // `DATASET/NAME`
const std::string instanceName = splits.at(2);
- const auto description = queryDescription(name, timestamp);
+ const auto description = queryDescription(className, timestamp);
if (not description)
{
- ARMARX_WARNING << "Unknown object " << name;
+ ARMARX_WARNING << "Unknown object " << className;
return std::nullopt;
}
@@ -204,16 +204,14 @@ namespace armarx::armem::articulated_object
std::optional<robot::RobotDescription>
Reader::queryDescription(const std::string& name, const armem::Time& timestamp)
{
- // FIXME: why is `name` unused?
-
// Query all entities from provider.
armem::client::query::Builder qb;
// clang-format off
qb
.coreSegments().withName(properties.coreClassSegmentName)
- .providerSegments().withName(properties.providerName)
- .entities().all() // withName(name)
+ .providerSegments().all() //withName(properties.providerName)
+ .entities().withName(name)
.snapshots().latest();
// clang-format on
--
GitLab