Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Want to rewrite the micro_ros_subscriber for example_interfaces/msg/String. #1791

Open
drinkBr opened this issue Jun 20, 2024 · 50 comments
Open

Comments

@drinkBr
Copy link

drinkBr commented Jun 20, 2024

Hi,
I want to rewrite the micro_ros_subscriber for example_interfaces/msg/String.
However, when I tried to modify it myself, I encountered an error and couldn't proceed any further. I would like to know how to resolve this issue.

The operating environment
・ArduinoIDE2.1.1
・Arduino Due

program
<

#include <micro_ros_arduino.h>

#include <stdio.h>
#include <rcl/rcl.h>
#include <rcl/error_handling.h>
#include <rclc/rclc.h>
#include <rclc/executor.h>

#include <example_interfaces/msg/string.h>

rcl_subscription_t subscriber;
example_interfaces__msg__String msg;
rclc_executor_t executor;
rclc_support_t support;
rcl_allocator_t allocator;
rcl_node_t node;
rcl_timer_t timer;

#define LED_PIN 13

#define RCCHECK(fn) { rcl_ret_t temp_rc = fn; if((temp_rc != RCL_RET_OK)){error_loop();}}
#define RCSOFTCHECK(fn) { rcl_ret_t temp_rc = fn; if((temp_rc != RCL_RET_OK)){}}


void error_loop(){
  while(1){
    digitalWrite(LED_PIN, !digitalRead(LED_PIN));
    delay(100);
  }
}

void subscription_callback(const void * msgin)
{  
  const example_interfaces__msg__String * msg = (const example_interfaces__msg__String *)msgin;
  digitalWrite(LED_PIN, (msg->data = "0") ? LOW : HIGH);  
}

void setup() {
  set_microros_transports();
  
  pinMode(LED_PIN, OUTPUT);
  digitalWrite(LED_PIN, HIGH);  
  
  delay(2000);

  allocator = rcl_get_default_allocator();

  //create init_options
  RCCHECK(rclc_support_init(&support, 0, NULL, &allocator));

  // create node
  RCCHECK(rclc_node_init_default(&node, "micro_ros_arduino_node", "", &support));

  // create subscriber
  RCCHECK(rclc_subscription_init_default(
    &subscriber,
    &node,
    ROSIDL_GET_MSG_TYPE_SUPPORT(example_interfaces, msg, String),
    "key_hit_event"));

  // create executor
  RCCHECK(rclc_executor_init(&executor, &support.context, 1, &allocator));
  RCCHECK(rclc_executor_add_subscription(&executor, &subscriber, &msg, &subscription_callback, ON_NEW_DATA));
}

void loop() {
  delay(100);
  RCCHECK(rclc_executor_spin_some(&executor, RCL_MS_TO_NS(100)));
}

Error message

rror: no match for 'operator=' (operand types are 'const rosidl_runtime_c__String' and 'const char [2]')
   digitalWrite(LED_PIN, (msg->data = "0") ? LOW : HIGH);  
                                    ^

Thank you.

@hippo5329
Copy link

Is this a school assignment? You will need to solve it yourself.

Google "c++ string compare"

@drinkBr
Copy link
Author

drinkBr commented Jun 20, 2024

Hi,hippo5329

Thank you for reply.
It's not school assignment ,but, I am a beginner with Arduino and ROS2, so this might be a simple question.
I thought that setting msg->data = "\0" would work, but what do you think?

My concern is not just about the syntax. In msg->data, msg is example_interfaces__msg__String, but data was rosidl_runtime_c__String. Since data was defined as String data in example_interfaces__msg__String, I am puzzled as to why data is shown as belonging to a different class. I would appreciate your advice.

@hippo5329
Copy link

digitalWrite(LED_PIN, (msg->data == "0") ? LOW : HIGH);

The c++ string compare operator is "==" .

@hippo5329
Copy link

hippo5329 commented Jun 20, 2024

Welcome to ROS2/mciro-ROS. There are a lot more puzzles. Enjoy. :)

As I suggested. It will save a lot of you if you install Ubuntu 22.04 PC, platformio/vscode, esp32 and follow my wiki. Workflow 1 and 2, should get you ready to explore the ROS world. A bare esp32 module is enough. You don't need to build a robot immediately.

@hippo5329
Copy link

Although it is possible to run ROS2 on Windows, it is more difficult for a beginner. You should get a spare laptop/desktop to install Ubuntu 22.04 and ROS2 humble.

@hippo5329
Copy link

An old PC with 8GB memory will be fine. I used 10 years old PC myself. If you want to purchase new PC, be sure to have 32GB memory to build moveit2.

@drinkBr
Copy link
Author

drinkBr commented Jun 20, 2024

Hey, hippo5329
Now I use ROS2 in Ubuntu22.04 on Raspberry Pi 4B.But, when I ran that program, used Arduino IDE on WindowsPC.
Should I ran that program on Arduino IDE in Ubuntu?

@hippo5329
Copy link

I would not suggest Arduino IDE. It is very limited on Pi. And it is awkward to build static micro-ros library on Arduino IDE. I would suggest install platformio/vscode on your windows pc. It should work. I switched to Linux more than 30 years ago, and do not use windows since then. You should really setup an Ubuntu PC to learn and use ROS.

@hippo5329
Copy link

Due has very limited memory. It uses very low memory meta. I would not recommend Due.

@hippo5329
Copy link

I fired up a win11 vm and realized micro ros platformio won't work.

You can install platformio cli on pi to build the micro-ros firmware.

platfromio.ini

[env:due]
platform = atmelsam
board = due
framework = arduino
board_microros_distro = humble
lib_deps =
    https://github.com/micro-ROS/micro_ros_platformio

src/subcriber_twist.ino

#include <arduino.h>
#include <micro_ros_arduino.h>
...
void setup() {
  Serial.begin(115200);
  set_microros_serial_transports(Serial);
  // set_microros_transports();
  pinMode(LED_PIN, OUTPUT);
  digitalWrite(LED_PIN, HIGH);  
...

@drinkBr
Copy link
Author

drinkBr commented Jun 21, 2024

Do you mean downloading PlatformIO on Ubuntu running on a Raspberry Pi, instead of on Windows?

@hippo5329
Copy link

Yes. Please follow my wiki to install platformio cli to pi. You may skip the linorobot2 and linorobot2_hardware installation.

I have created a repo for the example. You may clone and tryout.

git clone https://github.com/hippo5329/micro-ros_subscriber_twist.git
cd micro-ros_subscriber_twist
pio run -t upload

@hippo5329
Copy link

You may check the commit to understand the conversion.

https://github.com/hippo5329/micro-ros_subscriber_twist/commit/1018b3975b5abb7ba21a94b9e1819132da4aa1d2

@hippo5329
Copy link

However, it will save some efforts in the future if you install the two lino projects.

@hippo5329
Copy link

Due has very limited memory. You should get some esp32. It has 5 times the RAM and 8 times the Flash ROM of Due. The CPU speed is much faster on esp32, too. The WifI support of esp32 is another plus.

@drinkBr
Copy link
Author

drinkBr commented Jun 21, 2024

When I executed 'pio run -t upload', the following error occurred.

Command 'pio' not found, but can be installed with:
sudo apt install platformio

The actions I took were:
curl -fsSL -o get-platformio.py https://raw.githubusercontent.com/platformio/platformio-core-installer/master/get-platformio.py python3 get-platformio.py curl -fsSL https://raw.githubusercontent.com/platformio/platformio-core/develop/platformio/assets/system/99-platformio-udev.rules | sudo tee /etc/udev/rules.d/99-platformio-udev.rules sudo service udev restart sudo usermod -a -G dialout $USER sudo usermod -a -G plugdev $USER git clone https://github.com/hippo5329/micro-ros_subscriber_twist.git cd micro-ros_subscriber_twist pio run -t upload
Is it okay to proceed with this?

@hippo5329
Copy link

Please complete the software installation in my wiki. Do not skip any steps.

The steps to install platformio as in my wiki.

sudo apt remove brltty -y
sudo apt install python3-venv build-essential cmake git curl -y

curl -fsSL -o get-platformio.py https://raw.githubusercontent.com/platformio/platformio-core-installer/master/get-platformio.py
python3 get-platformio.py

echo "PATH="$PATH:$HOME/.platformio/penv/bin"" >> ~/.bashrc
source ~/.bashrc

curl -fsSL https://raw.githubusercontent.com/platformio/platformio-core/develop/platformio/assets/system/99-platformio-udev.rules | sudo tee /etc/udev/rules.d/99-platformio-udev.rules
sudo service udev restart
sudo usermod -a -G dialout $USER
sudo usermod -a -G plugdev $USER

@hippo5329
Copy link

You may be confused in the beginning. But in the future, you will need to understand what and why every steps in my wiki.

@hippo5329
Copy link

Updated. Please check README.

https://github.com/hippo5329/micro-ros_subscriber_twist

@hippo5329
Copy link

hippo5329 commented Jun 22, 2024

My previous replies on string msg are wrong.

String is complex datatype and needs special memory allocation. Please check this,
https://github.com/hippo5329/micro-ROS-demos-platformio/blob/rolling/rclc/string_subscriber/src/main.c

digitalWrite(LED_PIN, strcmp(msg->data.data, "0") ? HIGH : LOW);

There are more examples in micro_ros_demos. You may clone my repo and try out.

https://github.com/hippo5329/micro-ROS-demos-platformio

I got a Due and gave it 20 mins try-out with both int32_publisher and string_publisher. I am not able to get it connect to mico-ROS after upload. I give up because I am not interested in Due.

I have tested string_publisher and string_subscriber on two esp32_wifi. They work well.

I have heard: "Hello from micro-ROS #5"
I have heard: "Hello from micro-ROS #6"
I have heard: "Hello from micro-ROS #7"

@drinkBr
Copy link
Author

drinkBr commented Jun 22, 2024

To install platformio,I followed all the steps, but the following error occurred.

Command 'pio' not found, but can be installed with:
sudo apt install platformio

"After running sudo apt install platformio, it seemed to work. When I ran pio run -e due -t upload, the following result occurred."

Processing due (platform: atmelsam; board: due; framework: arduino)
--------------------------------------------------------------------------------
PlatformManager: Installing atmelsam
Error: Detected unknown package 'atmelsam'

Additional note: I am considering purchasing an ESP32.

@hippo5329
Copy link

hippo5329 commented Jun 22, 2024

Did you create the sd image with rpi-imager? Which OS version did you choose? You should use ubuntu 22.04 lts 64 bits desktop or server (better). Do not use core version.

You may use ubuntu lts 24.04 and ROS2 jazzy. It will get better support in the future.

@hippo5329
Copy link

Please check your platformio installation with

ls ~/.platformio/penv/bin

If you can find pio file here, then it is problem in the PATH. Try

echo $PATH

You should find the above path included.

@hippo5329
Copy link

I checked again on a rpi3b ubuntu 22.04 lts server 64 bits. There is no issue in pio installation.

https://github.com/hippo5329/micro-ROS-demos-platformio#install-platformio

@hippo5329
Copy link

I forced pushed update. Due should work with most example packages now. Please remove old dir and clone again. Some won't work due to Due limitation.

@hippo5329
Copy link

The arduino sam port has not been updated for 4 years. Issues remains unresolved. Arduino Due should be avoided.

@drinkBr
Copy link
Author

drinkBr commented Jun 25, 2024

I'm sorry for the late reply.

Which OS version did you choose?

I chose Ubuntu Desktop 22.04 LTS(64bit). So, I used ROS2 humble.

Please check your platformio installation with
ls ~/.platformio/penv/bin
If you can find pio file here, then it is problem in the PATH. Try
echo $PATH
You should find the above path included.

The results of the execution are as follows. I have omitted the description regarding user information.

$ls ~/.platformio/penv/bin
activate               bottle.py    pip3               pyserial-ports  tabulate
activate.csh           normalizer   pip3.10            python          uvicorn
activate.fish          pio          platformio         python3
Activate.ps1           piodebuggdb  __pycache__        python3.10
async-json-rpc-server  pip          pyserial-miniterm  readelf.py
$ echo $PATH
/opt/ros/humble/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin:/snap/bin:~/.platformio/penv/bin:~/.platformio/penv/bin:~/.platformio/penv/bin

What should I do? Should I start over from the beginning with the PlatformIO procedures?

Also, I got ESP32. It says ESP-WROOM-32, but is it compatible with your wiki?

@drinkBr
Copy link
Author

drinkBr commented Jun 25, 2024

Also, I have one more question.
In micro_ros_setup, In my case, I am not sure what I should execute due to its complexity. Please advise.

@hippo5329
Copy link

The PATH setup was wrong. Please edit your .bashrc, remove all lines containing "~/.platformio/penv/bin" . Then add this line to the end,

PATH="$PATH:$HOME/.platformio/penv/bin"

Then open a new terminal, and check with, "which pio". It should be like "/home/ubuntu/.platformio/penv/bin/pio" , here ubuntu is your user name. The pio command should be available now.

The ESP-WROOM-32 works well.

If you follow my wiki, no need to worry about micro ros setup. It is included in the linorobot2 installation. The micro-ros agent will be built and installed. You can follow the steps up to check topics with a WROOM esp32 module and wfi transport. This is a setup to help you learn micro-ros. You do not need to use the firmware in the future. Wifi transport is very helpful to run the various micro-ros demos.

https://github.com/hippo5329/linorobot2_hardware/wiki#install-linorobot2
https://github.com/hippo5329/linorobot2_hardware/wiki#install-firmware-source
https://github.com/hippo5329/linorobot2_hardware/wiki#esp32-with-micro-ros-serial-transport-on-devttyusb0
https://github.com/hippo5329/linorobot2_hardware/wiki#check-topics
https://github.com/hippo5329/linorobot2_hardware/wiki#esp32-with-micro-ros-wifi-transport

@hippo5329
Copy link

For your information,

https://github.com/hippo5329/linorobot2_hardware/wiki#build-micro-ros-agent-alone---optional

You do not need this. The linorobot2 installation will build micro-ROS agent.

@drinkBr
Copy link
Author

drinkBr commented Jun 27, 2024

Please edit your .bashrc, remove all lines containing "~/.platformio/penv/bin" . Then add this line to the end,

PATH="$PATH:$HOME/.platformio/penv/bin"
Then open a new terminal, and check with, "which pio". It should be like "/home/ubuntu/.platformio/penv/bin/pio" , here ubuntu is your user name.

After executing this, the results of executing steps pio run -e due -t upload and pio run -e esp32 -t upload are as follows.

$ which pio
/usr/bin/pio
$ pio run -e due -t upload
Processing due (platform: atmelsam; board: due; framework: arduino)
--------------------------------------------------------------------------------------------------
PlatformManager: Installing atmelsam
Error: Detected unknown package 'atmelsam'
$ pio run -e esp32 -t upload
Processing esp32 (platform: espressif32; board: nodemcu-32s; framework: arduino)
--------------------------------------------------------------------------------------------------
PlatformManager: Installing espressif32
Error: Detected unknown package 'espressif32'

Is PlatformIO functioning correctly with this? Should I start over from the beginning?

@hippo5329
Copy link

It is because you installed platformio with apt, which is /usr/bin/pio. While we install platformio with python penv. You may try "sudo apt remove platformio". Or you may start over with a fresh ubuntu 22.04. You will need to execute those commands I listed in my wiki. No need to follow the links to upstream sites. I should remove those link to external sites, such as platformio. It causes confusion.

@hippo5329
Copy link

I have removed those confusing external links. You may start here on a fresh ubuntu 22.04 rpi4,

https://github.com/hippo5329/linorobot2_hardware/wiki#install-the-software

@drinkBr
Copy link
Author

drinkBr commented Jun 27, 2024

After executing sudo apt remove platformio, I executed pio run -e esp32 -t upload, and the results are as follows.

$ pio run -e esp32 -t upload
Processing esp32 (platform: espressif32; board: nodemcu-32s; framework: arduino)
--------------------------------------------------------------------------------------------------
Verbose mode can be enabled via `-v, --verbose` option
CONFIGURATION: https://docs.platformio.org/page/boards/espressif32/nodemcu-32s.html
PLATFORM: Espressif 32 (6.7.0) > NodeMCU-32S
HARDWARE: ESP32 240MHz, 320KB RAM, 4MB Flash
...
Leaving...
Hard resetting via RTS pin...
================================== [SUCCESS] Took 64.41 seconds ==================================

Environment    Status    Duration
-------------  --------  ------------
esp32          SUCCESS   00:01:04.410

Is this successful?

@drinkBr
Copy link
Author

drinkBr commented Jun 27, 2024

How should I write 'cd firmware' when moving from the home directory as described in 'esp32 with micro-ROS serial transport on /dev/ttyUSB0'?

https://github.com/hippo5329/linorobot2_hardware/wiki#esp32-with-micro-ros-serial-transport-on-devttyusb0

@hippo5329
Copy link

Good. The esp32 built and uploaded successfully. You may proceed to test micro ROS connection.

cd linorobot2_hardware/firmware
pio run -e esp32 -t upload
ros2 run micro_ros_agent micro_ros_agent serial --dev /dev/ttyUSB0 --baudrate 921600

@drinkBr
Copy link
Author

drinkBr commented Jun 28, 2024

After executing step 1, the following message was displayed

$ ros2 run micro_ros_agent micro_ros_agent serial --dev /dev/ttyUSB0 --baudrate 921600
[1719542435.823438] info     | TermiosAgentLinux.cpp | init                     | running...             | fd: 3
[1719542435.824400] info     | Root.cpp           | set_verbose_level        | logger setup           | verbose_level: 4
[1719542435.861665] info     | Root.cpp           | create_client            | create                 | client_key: 0x1584DFE1, session_id: 0x81
[1719542435.861839] info     | SessionManager.hpp | establish_session        | session established    | client_key: 0x1584DFE1, address: 0
[1719542435.909273] info     | ProxyClient.cpp    | create_participant       | participant created    | client_key: 0x1584DFE1, participant_id: 0x000(1)
[1719542435.912894] info     | ProxyClient.cpp    | create_topic             | topic created          | client_key: 0x1584DFE1, topic_id: 0x000(2), participant_id: 0x000(1)
[1719542435.916033] info     | ProxyClient.cpp    | create_publisher         | publisher created      | client_key: 0x1584DFE1, publisher_id: 0x000(3), participant_id: 0x000(1)
[1719542435.920754] info     | ProxyClient.cpp    | create_datawriter        | datawriter created     | client_key: 0x1584DFE1, datawriter_id: 0x000(5), publisher_id: 0x000(3)
[1719542435.923980] info     | ProxyClient.cpp    | create_topic             | topic created          | client_key: 0x1584DFE1, topic_id: 0x001(2), participant_id: 0x000(1)
[1719542435.926861] info     | ProxyClient.cpp    | create_publisher         | publisher created      | client_key: 0x1584DFE1, publisher_id: 0x001(3), participant_id: 0x000(1)
[1719542435.931228] info     | ProxyClient.cpp    | create_datawriter        | datawriter created     | client_key: 0x1584DFE1, datawriter_id: 0x001(5), publisher_id: 0x001(3)
[1719542435.934995] info     | ProxyClient.cpp    | create_topic             | topic created          | client_key: 0x1584DFE1, topic_id: 0x002(2), participant_id: 0x000(1)
[1719542435.937840] info     | ProxyClient.cpp    | create_publisher         | publisher created      | client_key: 0x1584DFE1, publisher_id: 0x002(3), participant_id: 0x000(1)
[1719542435.942201] info     | ProxyClient.cpp    | create_datawriter        | datawriter created     | client_key: 0x1584DFE1, datawriter_id: 0x002(5), publisher_id: 0x002(3)
[1719542435.946129] info     | ProxyClient.cpp    | create_topic             | topic created          | client_key: 0x1584DFE1, topic_id: 0x003(2), participant_id: 0x000(1)
[1719542435.949351] info     | ProxyClient.cpp    | create_subscriber        | subscriber created     | client_key: 0x1584DFE1, subscriber_id: 0x000(4), participant_id: 0x000(1)
[1719542435.958817] info     | ProxyClient.cpp    | create_datareader        | datareader created     | client_key: 0x1584DFE1, datareader_id: 0x000(6), subscriber_id: 0x000(4)

Is this successful?
I have more question about commands in Terminal.
I think pio run -e esp32 -t upload writes the program to the ESP32. ros2 run micro_ros_agent micro_ros_agent serial --dev /dev/ttyUSB0 --baudrate 921600 connects the ESP32 to ROS2.
This is what I think; is it correct?

Also, I believe the program written in pio run -e esp32 -t upload is firmware.cpp located in ~/linorobot2_hardware/firmware/src. Is it possible to replace this with another program, such as micro_ros_subscriber_twist.ino? The content of firmware.cpp is very complex and difficult for me to understand, so I would like to know what each phase is doing.

One more question please.
.cpp files can be edited with a text editor, but .ino files can currently only be edited with the Arduino IDE. Is this okay?

@hippo5329
Copy link

hippo5329 commented Jun 28, 2024

After executing step 1, the following message was displayed
Is this successful? I have more question about commands in Terminal. I think pio run -e esp32 -t upload writes the program to the ESP32. ros2 run micro_ros_agent micro_ros_agent serial --dev /dev/ttyUSB0 --baudrate 921600 connects the ESP32 to ROS2. This is what I think; is it correct?

It is successful.
Correct.

Also, I believe the program written in pio run -e esp32 -t upload is firmware.cpp located in ~/linorobot2_hardware/firmware/src. Is it possible to replace this with another program, such as micro_ros_subscriber_twist.ino? The content of firmware.cpp is very complex and difficult for me to understand, so I would like to know what each phase is doing.

You should proceed to check to topics. Google the massages types. Make sure you understand the commands and messages.
https://github.com/hippo5329/linorobot2_hardware/wiki#check-topics

Then proceed to wifi transport. Connect via wifi transport and check topic again. You should learn wifi tansport because serial transport will occupy the serial port, and you cannot use Serial.print() or printf(). You should use wifi transport so that you can use "pio device monitor ..." command to see the serial output.
https://github.com/hippo5329/linorobot2_hardware/wiki#esp32-with-micro-ros-wifi-transport

After checked topics and wifi transport, you may leave the linorobot2_hardware. And proceed to other examples.

  1. the ported subscribe twist example from micro_ros_arduino.
    https://github.com/hippo5329/micro-ros_subscriber_twist

  2. the ported micro_ros_demos.
    https://github.com/hippo5329/micro-ROS-demos-platformio/wiki

One more question please. .cpp files can be edited with a text editor, but .ino files can currently only be edited with the Arduino IDE. Is this okay?

A .ino file can be edited with any text editor. In file manager, select file, right mouse click, "open with" Text Editor.
Or you may install gedit, an easy-to-use and general-purpose text editor.

sudo apt install -y gedit

@hippo5329
Copy link

OR you may install vscode on rpi4. Or install vscode on windows, and use remote development extension ssh into pi.

@drinkBr
Copy link
Author

drinkBr commented Jun 29, 2024

You should proceed to check to topics. Google the massages types. Make sure you understand the commands and messages.
https://github.com/hippo5329/linorobot2_hardware/wiki#check-topics

I checked topics. They're correct.

  1. the ported subscribe twist example from micro_ros_arduino. https://github.com/hippo5329/micro-ros_subscriber_twist

This did not work, and the following error was found.

Linking .pio/build/esp32/firmware.elf
~/.platformio/packages/toolchain-xtensa-esp32/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld: .pio/build/esp32/libFrameworkArduino.a(main.cpp.o):(.literal._Z8loopTaskPv+0x8): undefined reference to `setup()'
~/.platformio/packages/toolchain-xtensa-esp32/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld: .pio/build/esp32/libFrameworkArduino.a(main.cpp.o):(.literal._Z8loopTaskPv+0xc): undefined reference to `loop()'
~/.platformio/packages/toolchain-xtensa-esp32/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld: .pio/build/esp32/libFrameworkArduino.a(main.cpp.o): in function `loopTask(void*)':
~/.platformio/packages/framework-arduinoespressif32/cores/esp32/main.cpp:42: undefined reference to `setup()'
~/.platformio/packages/toolchain-xtensa-esp32/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld: /home/agri/.platformio/packages/framework-arduinoespressif32/cores/esp32/main.cpp:48: undefined reference to `loop()'
collect2: error: ld returned 1 exit status
*** [.pio/build/esp32/firmware.elf] Error 1
===================================== [FAILED] Took 31.65 seconds =====================================

Environment    Status    Duration
-------------  --------  ------------
esp32          FAILED    00:00:31.650

It seems like it doesn't recognize setup() and loop().
I changed the baud rate of micro-ros_subscriber_twist.ino to 921600, but it did not resolve the issue. This is probably unrelated to the issue.
I think the cause of this issue is as follows.

  1. The code to be uploaded is not in /linorobot2_hardware.
  2. The file format is .ino.
  3. The micro-ros_subscriber_twist I am using was cloned from git last week.

I would like to hear your thoughts.

@hippo5329
Copy link

You should proceed to check to topics. Google the massages types. Make sure you understand the commands and messages.
https://github.com/hippo5329/linorobot2_hardware/wiki#check-topics

I checked topics. They're correct.

Good.

  1. the ported subscribe twist example from micro_ros_arduino. https://github.com/hippo5329/micro-ros_subscriber_twist

This did not work, and the following error was found.

Linking .pio/build/esp32/firmware.elf
~/.platformio/packages/toolchain-xtensa-esp32/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld: .pio/build/esp32/libFrameworkArduino.a(main.cpp.o):(.literal._Z8loopTaskPv+0x8): undefined reference to `setup()'
~/.platformio/packages/toolchain-xtensa-esp32/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld: .pio/build/esp32/libFrameworkArduino.a(main.cpp.o):(.literal._Z8loopTaskPv+0xc): undefined reference to `loop()'
~/.platformio/packages/toolchain-xtensa-esp32/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld: .pio/build/esp32/libFrameworkArduino.a(main.cpp.o): in function `loopTask(void*)':
~/.platformio/packages/framework-arduinoespressif32/cores/esp32/main.cpp:42: undefined reference to `setup()'
~/.platformio/packages/toolchain-xtensa-esp32/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld: /home/agri/.platformio/packages/framework-arduinoespressif32/cores/esp32/main.cpp:48: undefined reference to `loop()'
collect2: error: ld returned 1 exit status
*** [.pio/build/esp32/firmware.elf] Error 1
===================================== [FAILED] Took 31.65 seconds =====================================

Environment    Status    Duration
-------------  --------  ------------
esp32          FAILED    00:00:31.650

It seems like it doesn't recognize setup() and loop(). I changed the baud rate of micro-ros_subscriber_twist.ino to 921600, but it did not resolve the issue. This is probably unrelated to the issue. I think the cause of this issue is as follows.

1. The code to be uploaded is not in /linorobot2_hardware.

It is correct. It is not related to linorobot2_hardware. It should be outside linorobot2_hardware.

2. The file format is .ino.

It does not matter the file extension is .ino or .cpp .

3. The micro-ros_subscriber_twist I am using was cloned from git last week.

Please remove the old clone directory and clone again.

I would like to hear your thoughts.

Building micro-ROS library
Found 34 compatible libraries
Scanning dependencies...
Dependency Graph
|-- micro_ros_platformio @ 0.0.1+sha.25ab659
Building in release mode
Compiling .pio/build/esp32/src/micro-ros_subscriber_twist.ino.cpp.o
Building .pio/build/esp32/bootloader.bin
Generating partitions .pio/build/esp32/partitions.bin
Compiling .pio/build/esp32/libe28/micro_ros_platformio/platform_code/arduino/clock_gettime.cpp.o
esptool.py v4.5.1
Creating esp32 image...
Merged 1 ELF section
Successfully created esp32 image.
Compiling .pio/build/esp32/libe28/micro_ros_platformio/platform_code/arduino/serial/micro_ros_transport.cpp.o
Compiling .pio/build/esp32/FrameworkArduino/Esp.cpp.o
Compiling .pio/build/esp32/FrameworkArduino/FirmwareMSC.cpp.o
Compiling .pio/build/esp32/FrameworkArduino/FunctionalInterrupt.cpp.o
Compiling .pio/build/esp32/FrameworkArduino/HWCDC.cpp.o
Compiling .pio/build/esp32/FrameworkArduino/HardwareSerial.cpp.o
Archiving .pio/build/esp32/libe28/libmicro_ros_platformio.a
Indexing .pio/build/esp32/libe28/libmicro_ros_platformio.a
Compiling .pio/build/esp32/FrameworkArduino/IPAddress.cpp.o
Compiling .pio/build/esp32/FrameworkArduino/IPv6Address.cpp.o
Compiling .pio/build/esp32/FrameworkArduino/MD5Builder.cpp.o
Compiling .pio/build/esp32/FrameworkArduino/Print.cpp.o
Compiling .pio/build/esp32/FrameworkArduino/Stream.cpp.o
Compiling .pio/build/esp32/FrameworkArduino/StreamString.cpp.o
Compiling .pio/build/esp32/FrameworkArduino/Tone.cpp.o
Compiling .pio/build/esp32/FrameworkArduino/USB.cpp.o
Compiling .pio/build/esp32/FrameworkArduino/USBCDC.cpp.o
Compiling .pio/build/esp32/FrameworkArduino/USBMSC.cpp.o
Compiling .pio/build/esp32/FrameworkArduino/WMath.cpp.o
Compiling .pio/build/esp32/FrameworkArduino/WString.cpp.o
Compiling .pio/build/esp32/FrameworkArduino/base64.cpp.o
Compiling .pio/build/esp32/FrameworkArduino/cbuf.cpp.o
Compiling .pio/build/esp32/FrameworkArduino/esp32-hal-adc.c.o
Compiling .pio/build/esp32/FrameworkArduino/esp32-hal-bt.c.o
Compiling .pio/build/esp32/FrameworkArduino/esp32-hal-cpu.c.o
Compiling .pio/build/esp32/FrameworkArduino/esp32-hal-dac.c.o
Compiling .pio/build/esp32/FrameworkArduino/esp32-hal-gpio.c.o
Compiling .pio/build/esp32/FrameworkArduino/esp32-hal-i2c-slave.c.o
Compiling .pio/build/esp32/FrameworkArduino/esp32-hal-i2c.c.o
Compiling .pio/build/esp32/FrameworkArduino/esp32-hal-ledc.c.o
Compiling .pio/build/esp32/FrameworkArduino/esp32-hal-matrix.c.o
Compiling .pio/build/esp32/FrameworkArduino/esp32-hal-misc.c.o
Compiling .pio/build/esp32/FrameworkArduino/esp32-hal-psram.c.o
Compiling .pio/build/esp32/FrameworkArduino/esp32-hal-rgb-led.c.o
Compiling .pio/build/esp32/FrameworkArduino/esp32-hal-rmt.c.o
Compiling .pio/build/esp32/FrameworkArduino/esp32-hal-sigmadelta.c.o
Compiling .pio/build/esp32/FrameworkArduino/esp32-hal-spi.c.o
Compiling .pio/build/esp32/FrameworkArduino/esp32-hal-time.c.o
Compiling .pio/build/esp32/FrameworkArduino/esp32-hal-timer.c.o
Compiling .pio/build/esp32/FrameworkArduino/esp32-hal-tinyusb.c.o
Compiling .pio/build/esp32/FrameworkArduino/esp32-hal-touch.c.o
Compiling .pio/build/esp32/FrameworkArduino/esp32-hal-uart.c.o
Compiling .pio/build/esp32/FrameworkArduino/firmware_msc_fat.c.o
Compiling .pio/build/esp32/FrameworkArduino/libb64/cdecode.c.o
Compiling .pio/build/esp32/FrameworkArduino/libb64/cencode.c.o
Compiling .pio/build/esp32/FrameworkArduino/main.cpp.o
Compiling .pio/build/esp32/FrameworkArduino/stdlib_noniso.c.o
Compiling .pio/build/esp32/FrameworkArduino/wiring_pulse.c.o
Compiling .pio/build/esp32/FrameworkArduino/wiring_shift.c.o
Archiving .pio/build/esp32/libFrameworkArduino.a
Indexing .pio/build/esp32/libFrameworkArduino.a
Linking .pio/build/esp32/firmware.elf
Retrieving maximum program size .pio/build/esp32/firmware.elf
Checking size .pio/build/esp32/firmware.elf
Advanced Memory Usage is available via "PlatformIO Home > Project Inspect"
RAM:   [=         ]  12.4% (used 40772 bytes from 327680 bytes)
Flash: [===       ]  25.6% (used 335649 bytes from 1310720 bytes)
Building .pio/build/esp32/firmware.bin
esptool.py v4.5.1
Creating esp32 image...
Merged 2 ELF sections
Successfully created esp32 image.
======================== [SUCCESS] Took 368.52 seconds ========================

Environment    Status    Duration
-------------  --------  ------------
esp32          SUCCESS   00:06:08.515
========================= 1 succeeded in 00:06:08.515 =========================

@hippo5329
Copy link

hippo5329 commented Jun 29, 2024

Please follow the new wiki. I added more arduino examples. The old micro-ros_subscriber_twist is removed.

https://github.com/hippo5329/micro_ros_arduino_examples_platformio/wiki

@drinkBr
Copy link
Author

drinkBr commented Jun 30, 2024

Linking .pio/build/esp32/firmware.elf
~/.platformio/packages/toolchain-xtensa-esp32/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld: .pio/build/esp32/libFrameworkArduino.a(main.cpp.o):(.literal._Z8loopTaskPv+0x8): undefined reference to `setup()'
~/.platformio/packages/toolchain-xtensa-esp32/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld: .pio/build/esp32/libFrameworkArduino.a(main.cpp.o):(.literal._Z8loopTaskPv+0xc): undefined reference to `loop()'
~/.platformio/packages/toolchain-xtensa-esp32/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld: .pio/build/esp32/libFrameworkArduino.a(main.cpp.o): in function `loopTask(void*)':
~/.platformio/packages/framework-arduinoespressif32/cores/esp32/main.cpp:42: undefined reference to `setup()'
~/.platformio/packages/toolchain-xtensa-esp32/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld: /home/agri/.platformio/packages/framework-arduinoespressif32/cores/esp32/main.cpp:48: undefined reference to `loop()'
collect2: error: ld returned 1 exit status
*** [.pio/build/esp32/firmware.elf] Error 1
===================================== [FAILED] Took 31.65 seconds =====================================

Environment    Status    Duration
-------------  --------  ------------
esp32          FAILED    00:00:31.650

It seems like it doesn't recognize setup() and loop().

When editing the .ino file using the Arduino IDE, a new folder is created in the location where the file was, and the edited file is saved in that folder. Is this okay? I believe this is the reason why setup() and loop() were not recognized.

@drinkBr
Copy link
Author

drinkBr commented Jun 30, 2024

As a test, I changed the LED_PIN in micro-ros_publisher to 4 and uploaded the program, but the attached LED did not light up. Why is this? The positive side of the LED is connected to "D4" and the negative side to "GND". The resistor is 1kΩ. I realized while writing that the resistor might be too large.

Also, should the pin numbers specified in the program be the same as the pin numbers written on the ESP32 itself? I researched this but couldn't find a clear answer.

@hippo5329
Copy link

hippo5329 commented Jun 30, 2024

When editing the .ino file using the Arduino IDE, a new folder is created in the location where the file was, and the edited file is saved in that folder. Is this okay? I believe this is the reason why setup() and loop() were not recognized.

So the arduino ide should not be used to edit this project. Arduino will interfere with our build process. Please install gedit and use gedit to edit your project. Remove the old directory and clone again.

You may remove arduino ide to avoid the interference.

@hippo5329
Copy link

hippo5329 commented Jun 30, 2024

As a test, I changed the LED_PIN in micro-ros_publisher to 4 and uploaded the program, but the attached LED did not light up. Why is this? The positive side of the LED is connected to "D4" and the negative side to "GND". The resistor is 1kΩ. I realized while writing that the resistor might be too large.

Also, should the pin numbers specified in the program be the same as the pin numbers written on the ESP32 itself? I researched this but couldn't find a clear answer.

I have changed the LED_PIN in my port.

#ifdef LED_BUILTIN
#define LED_PIN LED_BUILTIN
#else
#define LED_PIN 13
#endif

I used LED_BUILTIN, which is 2 in most esp32 modules, including wroom. Wroom has two LEDs on board. The red led is power. The blue led is 2. You should see blue led lights up.

If the micro-ros client times out, the led will blink quickly. Press the "EN" button near the usb header to reset. Then it should be able to connect to micro-ros agent.

@hippo5329
Copy link

A Blink LED example in platformio.

https://github.com/hippo5329/Blink-platformio

@hippo5329
Copy link

I think our conversations has gone far away from the scope of this repository. Please open a new issue in my repo,
https://github.com/hippo5329/micro_ros_arduino_examples_platformio/issues

@drinkBr
Copy link
Author

drinkBr commented Jun 30, 2024

I have created a new issue. Thank you.

hippo5329/micro_ros_arduino_examples_platformio#1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants