M1 digital input and output function works. (#64)

* M1 digital input and output function works.

* Update m1_real.launch

Changed can1 to can0 for consistency with the Getting Started Guide.

Co-authored-by: Justin <49131879+justincmfong@users.noreply.github.com>

Author: shortmr

.gitignore

@@ -9,4 +9,5 @@ CANopenSocket/canopend/X2_log.txt
 /build/*
 bbbxc
 EXO_APP_2020
-cmake-build-debug/
+cmake-build-debug/
+.idea/

CMakeLists.txt

@@ -14,10 +14,10 @@ project(CORC C CXX)
 
 set (STATE_MACHINE_NAME "ExoTestMachine")
 #set (STATE_MACHINE_NAME "M1DemoMachine")
-#set (STATE_MACHINE_NAME "M1DemoMachineROS")
+#set (STATE_MACHINE_NAME "M1DemoMachineROS") # uncomment other cfg files under generate_dynamic_reconfigure_options
 #set (STATE_MACHINE_NAME "M2DemoMachine")
 #set (STATE_MACHINE_NAME "M3DemoMachine")
-#set (STATE_MACHINE_NAME "X2DemoMachine")
+#set (STATE_MACHINE_NAME "X2DemoMachine")  # uncomment other cfg files under generate_dynamic_reconfigure_options
 #set (STATE_MACHINE_NAME "LoggingDevice")
 
 # Comment to use actual hardware, uncomment for a nor robot (virtual) app
@@ -136,7 +136,7 @@ if(USE_ROS)
 
         generate_dynamic_reconfigure_options(
                 config/m1_dynamic_params.cfg
-                config/x2_dynamic_params.cfg
+#                config/x2_dynamic_params.cfg
         )
 
         add_message_files(

launch/m1_real.launch

@@ -1,11 +1,12 @@
 <?xml version="1.0"?>
 <launch>
 
-    <arg name="robot_name_1" default="m1_x"/>
+    <arg name="robot_name_1" default="m1_y"/>
     <arg name="name_spaces" default="[$(arg robot_name_1)]"/>
 
     <!-- CORC M1_X -->
-    <node name="$(arg robot_name_1)" pkg="CORC" type="MultiM1Machine_APP" output="screen" args="-can can0"/>
+    <node name="$(arg robot_name_1)" pkg="CORC" type="M1DemoMachineROS_APP" output="screen" args="-can can0"/>
+
 
     <!-- rqt -->
     <node name="rqt" pkg="CORC" type="rqt.sh">

src/apps/M1DemoMachineROS/states/MultiControllerState.cpp

@@ -179,54 +179,40 @@ void MultiControllerState::during(void) {
         }
     }
     else if (controller_mode_ == 11){ // SEND HIGH
-//        std::cout<<"SET HIGH"<<std::endl;
         double time = std::chrono::duration_cast<std::chrono::milliseconds>(std::chrono::steady_clock::now() - time0).count()/1000.0;
 
-        if(robot_->getRobotName() == "m1_y"){
-            //std::cout<<"ROBOT Y"<<std::endl;
-            if(time > 2.0){
-                if (digitalInValue_ == 1) {
-                    digitalOutValue_ = 0;
-                    robot_->setDigitalOut(digitalOutValue_);
-                }
-            }
-            else if(time > 1.0){
-                if (digitalInValue_ == 0) {
-                    digitalOutValue_ = 1;
-                    robot_->setDigitalOut(digitalOutValue_);
-                }
+        if(time > 1.0){
+            if (digitalOutValue_ == 0) {
+                digitalOutValue_ = 1;
+                robot_->setDigitalOut(digitalOutValue_);
             }
         }
     }
     else if (controller_mode_ == 12){ // SEND LOW
-//        std::cout<<"SET LOW"<<std::endl;
         double time = std::chrono::duration_cast<std::chrono::milliseconds>(std::chrono::steady_clock::now() - time0).count()/1000.0;
 
-        if(robot_->getRobotName() == "m1_y"){
-            //std::cout<<"ROBOT Y"<<std::endl;
-            if(time > 1.0){
-                if (digitalInValue_ == 1) {
-                    digitalOutValue_ = 0;
-                    robot_->setDigitalOut(digitalOutValue_);
-                }
+        if(time > 1.0){
+            if (digitalOutValue_ == 1) {
+                digitalOutValue_ = 0;
+                robot_->setDigitalOut(digitalOutValue_);
             }
         }
     }
     else if (controller_mode_ == 13){ // SEND HIGH-LOW once
-
         double time = std::chrono::duration_cast<std::chrono::milliseconds>(std::chrono::steady_clock::now() - time0).count()/1000.0;
-        if(robot_->getRobotName() == "m1_y"){
-            //std::cout<<"ROBOT Y"<<std::endl;
-            if (time > 1.0) {
-                //std::cout<<"Trigger Sent"<<std::endl;
-                digitalOutValue_ = (digitalOutValue_ == 1) ? 0 : 1;
+
+        if(time > 2.0){
+            if (digitalOutValue_ == 1) {
+                digitalOutValue_ = 0;
+                robot_->setDigitalOut(digitalOutValue_);
+            }
+        }
+        else if(time > 1.0){
+            if (digitalOutValue_ == 0) {
+                digitalOutValue_ = 1;
                 robot_->setDigitalOut(digitalOutValue_);
-                time0 = std::chrono::steady_clock::now();
             }
         }
-    }
-    if(robot_->getRobotName() == "m1_y"){
-        digitalInValue_ = robot_->getDigitalIn();
     }
 }
 
@@ -267,6 +253,9 @@ void MultiControllerState::dynReconfCallback(CORC::dynamic_paramsConfig &config,
         if(controller_mode_ == 11) robot_->setDigitalOut(0);
         if(controller_mode_ == 12) time0 = std::chrono::steady_clock::now();
         if(controller_mode_ == 12) robot_->setDigitalOut(1);
+        if(controller_mode_ == 13) time0 = std::chrono::steady_clock::now();
+        if(controller_mode_ == 13) robot_->setDigitalOut(0);
+
     }
 
     return;

src/core/robot/Drive.h

@@ -295,8 +295,8 @@ class Drive {
         {TARGET_POS, 4},
         {TARGET_VEL, 4},
         {TARGET_TOR, 2},
-        {DIGITAL_IN, 4},
-        {DIGITAL_OUT, 4}};
+        {DIGITAL_IN, 2},
+        {DIGITAL_OUT, 2}};
 
     /**
      * \brief Map between the Commonly-used OD entries and their addresses and sub-index - used to generate PDO Configurations
@@ -343,8 +343,8 @@ class Drive {
     INTEGER32 targetPos=0;
     INTEGER32 targetVel=0;
     INTEGER16 targetTor=0;
-    UNSIGNED32 digitalIn=0;
-    UNSIGNED32 digitalOut=0;
+    UNSIGNED16 digitalIn=0;
+    UNSIGNED16 digitalOut=0;
     /**
      * \brief Current error state of the drive
      *

src/hardware/platforms/M1/RobotM1.cpp

@@ -35,6 +35,7 @@ RobotM1::RobotM1(string robot_name, string yaml_config_file):   Robot(robot_name
     posControlMotorProfile.profileDeceleration = 500.*65535*10000/4000000;
 
     //Check if YAML file exists and contain robot parameters
+    robotName_ = robot_name;
     initialiseFromYAML(yaml_config_file);
 
     initialiseJoints();
@@ -144,6 +145,7 @@ bool RobotM1::setDigitalOut(int digital_out) {
 }
 
 int RobotM1::getDigitalIn() {
+
     return ((JointM1 *)joints[0])->getDigitalIn();
 }