review applied

Signed-off-by: Adam Dabrowski <[email protected]>

Project/Gem/CMakeLists.txt

@@ -40,8 +40,6 @@ ly_add_target(
             Gem::Atom_AtomBridge.Static
 )
 
-#target_depends_on_ros2_packages(ROSConDemo.Static vision_msgs)
-
 ly_add_target(
     NAME ROSConDemo ${PAL_TRAIT_MONOLITHIC_DRIVEN_MODULE_TYPE}
     NAMESPACE Gem

Project/Gem/Source/ApplePicker/ApplePickerComponent.cpp

@@ -9,7 +9,6 @@
 #include "ApplePickerComponent.h"
 #include <AzCore/Serialization/EditContext.h>
 #include <AzCore/Serialization/EditContextConstants.inl>
-// #include <ROS2/ROS2Bus.h>
 
 namespace AppleKraken
 {
@@ -17,7 +16,7 @@ namespace AppleKraken
     {
         AZStd::string TaskString(const PickAppleTask& task)
         {
-            if (task.m_appleEntityId.IsValid())
+            if (!task.m_appleEntityId.IsValid())
             {
                 return "|Task for an unspecified apple|";
             }

Project/Gem/Source/ApplePicker/ApplePickerComponent.h

@@ -14,9 +14,6 @@
 
 namespace AppleKraken
 {
-    //! The component also acts as a ground-truth detector
-    // using DetectionsMessage = vision_msgs::msgs::Detection3DArray;
-
     //! Demo component handling orchestration of apple picking
     class ApplePickerComponent
         : public AZ::Component