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

[arci-ros2] Add arci-ros2 (only MoveBase trait) #291

Merged
merged 15 commits into from
May 20, 2021
Merged

[arci-ros2] Add arci-ros2 (only MoveBase trait) #291

merged 15 commits into from
May 20, 2021

Conversation

OTL
Copy link
Contributor

@OTL OTL commented Apr 27, 2021

It requires ROS2 Foxy (Ubuntu 20.04).

  • r2r is not published to crates.io yet.
  • It requires ROS2 foxy is installed.

@OTL OTL marked this pull request as draft April 27, 2021 09:44
@codecov
Copy link

codecov bot commented Apr 27, 2021

Codecov Report

Merging #291 (20db7c2) into main (0eadcbc) will increase coverage by 0.26%.
The diff coverage is 92.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #291      +/-   ##
==========================================
+ Coverage   42.57%   42.84%   +0.26%     
==========================================
  Files          80       82       +2     
  Lines        5085     5112      +27     
==========================================
+ Hits         2165     2190      +25     
- Misses       2920     2922       +2     
Impacted Files Coverage Δ
arci-ros2/src/cmd_vel_move_base.rs 66.66% <66.66%> (ø)
arci-ros2/tests/test_cmd_vel.rs 100.00% <100.00%> (ø)
arci/src/clients/dummy_navigation.rs 86.20% <0.00%> (-0.46%) ⬇️
arci/src/clients/joint_position_limiter.rs 90.38% <0.00%> (-0.27%) ⬇️
.../src/clients/joint_trajectory_clients_container.rs 92.41% <0.00%> (-0.07%) ⬇️
arci/tests/test_utils.rs 100.00% <0.00%> (ø)
arci/tests/test_waits.rs 100.00% <0.00%> (ø)
arci-ros/tests/test_joy.rs 92.50% <0.00%> (ø)
arci-urdf-viz/tests/test_web.rs 100.00% <0.00%> (ø)
arci-urdf-viz/tests/test_client.rs 100.00% <0.00%> (ø)
... and 6 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0eadcbc...20db7c2. Read the comment docs.

Comment on lines +30 to +32
source /opt/ros/foxy/setup.bash
# for test of arci-ros (roscore, rostopic)
source /opt/ros/melodic/setup.bash
cargo +nightly-2021-03-04 tarpaulin --verbose --all-features --workspace --out Xml
source /opt/ros/noetic/setup.bash
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

coverage on ubuntu20.04 (noetic/foxy) only.
melodic is excluded.

arci-ros2/Cargo.toml Show resolved Hide resolved
arci-ros2/tests/test_cmd_vel.rs Outdated Show resolved Hide resolved
arci-ros2/Cargo.toml Outdated Show resolved Hide resolved
OTL added a commit that referenced this pull request May 11, 2021
Co-authored-by: Taiki Endo <te316e89@gmail.com>
OTL added a commit that referenced this pull request May 11, 2021
Co-authored-by: Taiki Endo <te316e89@gmail.com>
@OTL
Copy link
Contributor Author

OTL commented May 14, 2021

r2r is broken now. I'm working on it.
sequenceplanner/r2r#10

@OTL
Copy link
Contributor Author

OTL commented May 17, 2021

r2r is fixed. They are using Rolling (the latest (unstable?) distro for ROS2) instead of foxy.

@OTL OTL merged commit 299732d into main May 20, 2021
@OTL OTL deleted the ros2-init branch May 20, 2021 04:20
OTL added a commit that referenced this pull request May 20, 2021
Co-authored-by: Taiki Endo <te316e89@gmail.com>
@taiki-e taiki-e mentioned this pull request Jun 21, 2021
3 tasks
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.

None yet

3 participants