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

fixed time.h import, added Arduino UNO R4 minima (USB), Arduino UNO R4 WiFi (USB and WiFi), Arduino Giga R1 (USB and WiFi) #1712

Merged
merged 19 commits into from
Nov 14, 2024

Conversation

gbr1
Copy link
Contributor

@gbr1 gbr1 commented Mar 21, 2024

By importing time.h instead of sys/time.h you can build micro-ros for Arduino UNO R4.

It addresses issue: #1461

This PR was tested on following targets:

  • mbed (Arduino GIGA R1)
  • m4 (Arduino UNO R4 Minima)
  • stm32 (STMicroelectronics Nucleo F401RE)

@gbr1 gbr1 mentioned this pull request Mar 21, 2024
@gbr1
Copy link
Contributor Author

gbr1 commented Mar 27, 2024

I added also Arduino Giga (USB and WiFi).

It is practically the same of an Arduino Portenta H7, so it was quite easy

@gbr1 gbr1 changed the title fixed time.h import fixed time.h import, added Arduino Giga R1 (USB and WiFi) Mar 27, 2024
@gbr1
Copy link
Contributor Author

gbr1 commented Mar 27, 2024

Latest PR adds WiFi for Arduino UNO R4.

@gbr1 gbr1 changed the title fixed time.h import, added Arduino Giga R1 (USB and WiFi) fixed time.h import, added Arduino UNO R4 minima (USB), Arduino UNO R4 WiFi (USB and WiFi), Arduino Giga R1 (USB and WiFi) Mar 27, 2024
@gbr1
Copy link
Contributor Author

gbr1 commented Jul 10, 2024

Hi @pablogs9 , any update on this PR?

@pablogs9
Copy link
Member

Could you please add the new supported boards to supported boards community table, also is there a possibility of having some CI on this new boards?

@pablogs9
Copy link
Member

CI failing

@gbr1
Copy link
Contributor Author

gbr1 commented Jul 11, 2024

@pablogs9 it should work now, let me know

@gbr1
Copy link
Contributor Author

gbr1 commented Jul 11, 2024

I don't understand how it works, I have modified only iron branch, should I change manually also rolling and humble?
Screenshot 2024-07-11 at 16 28 38

@pablogs9
Copy link
Member

I don't understand how it works,

How what works?

@gbr1
Copy link
Contributor Author

gbr1 commented Jul 11, 2024

I don't understand how it works,

How what works?

The CI

@pablogs9
Copy link
Member

@james-yoo
Copy link

Any update?

@gbr1
Copy link
Contributor Author

gbr1 commented Jul 24, 2024

@james-yoo I don't know why on my CI it works but fails on the one of microros :'(

@gbr1
Copy link
Contributor Author

gbr1 commented Aug 2, 2024

Hi @pablogs9 , it seems that everything is correct now

@gbr1
Copy link
Contributor Author

gbr1 commented Nov 14, 2024

@pablogs9 any update on this?

@pablogs9
Copy link
Member

Closing and reopening to trigger CI again

@pablogs9 pablogs9 closed this Nov 14, 2024
@pablogs9 pablogs9 reopened this Nov 14, 2024
@pablogs9
Copy link
Member

@mergify backport rolling jazzy humble

Copy link
Contributor

mergify bot commented Nov 14, 2024

@pablogs9
Copy link
Member

Sorry for the delay folks, merging it now. Thanks for the contribution.

@pablogs9 pablogs9 merged commit e347a4a into micro-ROS:iron Nov 14, 2024
3 checks passed
mergify bot pushed a commit that referenced this pull request Nov 14, 2024
…4 WiFi (USB and WiFi), Arduino Giga R1 (USB and WiFi) (#1712)

* fixes time.h import

* added support for Arduino GIGA R1 over WiFi

* examples with giga r1

* uint -> uint32_t, added uno r4 wifi

* added uno r4 into the example

* added uno r4 family into library properties

* led builtin check

* added boards into readme

* updated ci

* fix typo

* updated cli version

* force update index

* reverted cli

(cherry picked from commit e347a4a)

# Conflicts:
#	.github/workflows/ci.yml
#	README.md
#	examples/micro-ros_publisher_wifi/micro-ros_publisher_wifi.ino
#	examples/micro-ros_publisher_wifi_at/micro-ros_publisher_wifi_at.ino
#	src/micro_ros_arduino.h
#	src/wifi_transport.cpp
mergify bot pushed a commit that referenced this pull request Nov 14, 2024
…4 WiFi (USB and WiFi), Arduino Giga R1 (USB and WiFi) (#1712)

* fixes time.h import

* added support for Arduino GIGA R1 over WiFi

* examples with giga r1

* uint -> uint32_t, added uno r4 wifi

* added uno r4 into the example

* added uno r4 family into library properties

* led builtin check

* added boards into readme

* updated ci

* fix typo

* updated cli version

* force update index

* reverted cli

(cherry picked from commit e347a4a)

# Conflicts:
#	.github/workflows/ci.yml
#	README.md
#	examples/micro-ros_publisher_wifi/micro-ros_publisher_wifi.ino
#	examples/micro-ros_publisher_wifi_at/micro-ros_publisher_wifi_at.ino
#	src/micro_ros_arduino.h
#	src/wifi_transport.cpp
mergify bot pushed a commit that referenced this pull request Nov 14, 2024
…4 WiFi (USB and WiFi), Arduino Giga R1 (USB and WiFi) (#1712)

* fixes time.h import

* added support for Arduino GIGA R1 over WiFi

* examples with giga r1

* uint -> uint32_t, added uno r4 wifi

* added uno r4 into the example

* added uno r4 family into library properties

* led builtin check

* added boards into readme

* updated ci

* fix typo

* updated cli version

* force update index

* reverted cli

(cherry picked from commit e347a4a)
pablogs9 pushed a commit that referenced this pull request Nov 14, 2024
…4 WiFi (USB and WiFi), Arduino Giga R1 (USB and WiFi) (#1712) (#1865)

* fixes time.h import

* added support for Arduino GIGA R1 over WiFi

* examples with giga r1

* uint -> uint32_t, added uno r4 wifi

* added uno r4 into the example

* added uno r4 family into library properties

* led builtin check

* added boards into readme

* updated ci

* fix typo

* updated cli version

* force update index

* reverted cli

(cherry picked from commit e347a4a)

Co-authored-by: Giovanni Bruno <[email protected]>
@pablogs9
Copy link
Member

@gbr1 can you take a look at #1864 and #1863 ?

@gbr1
Copy link
Contributor Author

gbr1 commented Nov 14, 2024

It seems it wrongly detect arduino nano rp2040, but I am also trying to compile some firmware for other boards and it seems to be broken on rclc_timer_init_default declared in rclc/timer.h:63:1

@gbr1
Copy link
Contributor Author

gbr1 commented Nov 14, 2024

@pablogs9 is it correct that the ci test the build for raspberry pi pico (mbed_rp2040:pico) as target for publisher with wifi in AT mode? The board hasn't got the wifi

Please check here: https://github.com/micro-ROS/micro_ros_arduino/pull/1712/files#diff-b803fcb7f17ed9235f1e5cb1fcd2f5d3b2838429d4368ae4c57ce4436577f03fL86

@gbr1
Copy link
Contributor Author

gbr1 commented Nov 14, 2024

@pablogs9 I can confirm, it seems that raspberry pi pico target was added on BOARD_WITH_ESP_ATby this https://github.com/micro-ROS/micro_ros_arduino/pull/1425/commits but in jazzy seems that arduino-cli lib install WiFiEspATis missing in ci.yml because also building that example is removed

Iron one: https://github.com/micro-ROS/micro_ros_arduino/blob/iron/.github/workflows/ci.yml#L62C1-L63C1
Official Jazzy one: https://github.com/micro-ROS/micro_ros_arduino/blob/jazzy/.github/workflows/ci.yml#L51

Let me know how to proceed

pablogs9 added a commit that referenced this pull request Nov 15, 2024
…4 WiFi (USB and WiFi), Arduino Giga R1 (USB and WiFi) (backport #1712) (#1864)

* fixed time.h import, added Arduino UNO R4 minima (USB), Arduino UNO R4 WiFi (USB and WiFi), Arduino Giga R1 (USB and WiFi) (#1712)

* fixes time.h import

* added support for Arduino GIGA R1 over WiFi

* examples with giga r1

* uint -> uint32_t, added uno r4 wifi

* added uno r4 into the example

* added uno r4 family into library properties

* led builtin check

* added boards into readme

* updated ci

* fix typo

* updated cli version

* force update index

* reverted cli

(cherry picked from commit e347a4a)

# Conflicts:
#	.github/workflows/ci.yml
#	README.md
#	examples/micro-ros_publisher_wifi/micro-ros_publisher_wifi.ino
#	examples/micro-ros_publisher_wifi_at/micro-ros_publisher_wifi_at.ino
#	src/micro_ros_arduino.h
#	src/wifi_transport.cpp

* Fix rebase

* Update ci.yml

---------

Co-authored-by: Giovanni Bruno <[email protected]>
Co-authored-by: Pablo Garrido <[email protected]>
pablogs9 added a commit that referenced this pull request Nov 15, 2024
…4 WiFi (USB and WiFi), Arduino Giga R1 (USB and WiFi) (backport #1712) (#1863)

* fixed time.h import, added Arduino UNO R4 minima (USB), Arduino UNO R4 WiFi (USB and WiFi), Arduino Giga R1 (USB and WiFi) (#1712)

* fixes time.h import

* added support for Arduino GIGA R1 over WiFi

* examples with giga r1

* uint -> uint32_t, added uno r4 wifi

* added uno r4 into the example

* added uno r4 family into library properties

* led builtin check

* added boards into readme

* updated ci

* fix typo

* updated cli version

* force update index

* reverted cli

(cherry picked from commit e347a4a)

# Conflicts:
#	.github/workflows/ci.yml
#	README.md
#	examples/micro-ros_publisher_wifi/micro-ros_publisher_wifi.ino
#	examples/micro-ros_publisher_wifi_at/micro-ros_publisher_wifi_at.ino
#	src/micro_ros_arduino.h
#	src/wifi_transport.cpp

* Fix rebase

* Update ci.yml

---------

Co-authored-by: Giovanni Bruno <[email protected]>
Co-authored-by: Pablo Garrido <[email protected]>
@pablogs9
Copy link
Member

Got them merged, thanks a lot @gbr1

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

Successfully merging this pull request may close these issues.

4 participants