Skip to content

Commit

Permalink
Bump to YARP 3.8+
Browse files Browse the repository at this point in the history
  • Loading branch information
PeterBowman committed Mar 29, 2024
1 parent 61b8a44 commit 97ecca2
Show file tree
Hide file tree
Showing 7 changed files with 12 additions and 18 deletions.
14 changes: 7 additions & 7 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -35,9 +35,9 @@ jobs:
matrix:
os: [ubuntu-20.04, ubuntu-22.04]
robotology:
- { yarp: yarp-3.7, cmake: 3.16.x }
- { yarp: yarp-3.8, cmake: 3.16.x }
- { yarp: master, cmake: 3.16.x }
- { yarp: yarp-3.9, cmake: 3.16.x }
- { yarp: master, cmake: 3.19.x }
compiler:
- { cc: gcc, cxx: g++ }
- { cc: clang, cxx: clang++ }
Expand All @@ -48,23 +48,23 @@ jobs:

steps:
- name: Check out kinematics-dynamics
uses: actions/checkout@v3
uses: actions/checkout@v4

- name: Check out YCM
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
repository: robotology/ycm
path: .deps/ycm

- name: Check out YARP
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
repository: robotology/yarp
ref: ${{matrix.robotology.yarp}}
path: .deps/yarp

- name: Check out Orocos KDL
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
repository: orocos/orocos_kinematics_dynamics
path: .deps/kdl
Expand All @@ -73,7 +73,7 @@ jobs:
run: sudo apt-get install -qq ccache libeigen3-dev swig googletest

- name: Set up CMake
uses: jwlawson/actions-setup-cmake@v1
uses: jwlawson/actions-setup-cmake@v2
with:
cmake-version: ${{matrix.robotology.cmake}}

Expand Down
2 changes: 1 addition & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ list(APPEND CMAKE_MODULE_PATH ${CMAKE_SOURCE_DIR}/cmake/find-modules)

# Hard dependencies.
find_package(YCM 0.11 REQUIRED)
find_package(YARP 3.7 REQUIRED COMPONENTS os dev sig
find_package(YARP 3.8 REQUIRED COMPONENTS os dev sig
OPTIONAL_COMPONENTS math)

# Soft dependencies.
Expand Down
8 changes: 4 additions & 4 deletions doc/kinematics-dynamics-install.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,10 @@

First install the dependencies:

- [Install CMake 3.16+](https://github.com/roboticslab-uc3m/installation-guides/blob/master/docs/install-cmake.md/)
- [Install YCM 0.11+](https://github.com/roboticslab-uc3m/installation-guides/blob/master/docs/install-ycm.md/)
- [Install YARP 3.7+](https://github.com/roboticslab-uc3m/installation-guides/blob/master/docs/install-yarp.md/)
- [Install KDL 1.4+](https://github.com/roboticslab-uc3m/installation-guides/blob/master/docs/install-kdl.md/)
- [Install CMake 3.16+](https://github.com/roboticslab-uc3m/installation-guides/blob/master/install-cmake.md/)
- [Install YCM 0.11+](https://github.com/roboticslab-uc3m/installation-guides/blob/master/install-ycm.md/)
- [Install YARP 3.8+](https://github.com/roboticslab-uc3m/installation-guides/blob/master/install-yarp.md/)
- [Install KDL 1.4+](https://github.com/roboticslab-uc3m/installation-guides/blob/master/install-kdl.md/)

We optionally use the `ProximitySensorsClient` device from [yarp-devices](https://github.com/roboticslab-uc3m/yarp-devices) in the `haarDetectionController` app. Refer to its installation guide for instructions.

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,6 @@ bool BasicCartesianControl::open(yarp::os::Searchable& config)
yarp::os::Property robotOptions;
robotOptions.fromString(config.toString());
robotOptions.put("device", robotStr);
robotOptions.setMonitor(config.getMonitor(), robotStr.c_str());

if (!robotDevice.open(robotOptions))
{
Expand Down Expand Up @@ -190,8 +189,6 @@ bool BasicCartesianControl::open(yarp::os::Searchable& config)
yCInfo(BCC) << "Using joint limits provided via user configuration";
}

solverOptions.setMonitor(config.getMonitor(), solverStr.c_str());

if (!solverDevice.open(solverOptions))
{
yCError(BCC) << "Solver device not valid:" << solverStr;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,6 @@ bool CartesianControlServer::open(yarp::os::Searchable& config)
yarp::os::Property p;
p.fromString(config.toString());
p.put("device", name->toString());
p.setMonitor(config.getMonitor(), name->toString().c_str());
cartesianControlDevice.open(p);
}
else
Expand Down
1 change: 0 additions & 1 deletion libraries/YarpPlugins/KdlSolver/DeviceDriverImpl.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -164,7 +164,6 @@ bool KdlSolver::open(yarp::os::Searchable & config)
yarp::os::Property fullConfig;
fullConfig.fromConfigFile(kinematicsFullPath.c_str());
fullConfig.fromString(config.toString(), false);
fullConfig.setMonitor(config.getMonitor(), "KdlSolver");

yCDebug(KDLS) << "fullConfig:" << fullConfig.toString();

Expand Down
1 change: 0 additions & 1 deletion libraries/YarpPlugins/KdlTreeSolver/DeviceDriverImpl.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,6 @@ bool KdlTreeSolver::open(yarp::os::Searchable & config)
yarp::os::Property fullConfig;
fullConfig.fromConfigFile(kinematicsFullPath.c_str());
fullConfig.fromString(config.toString(), false);
fullConfig.setMonitor(config.getMonitor(), "KdlTreeSolver");

yCDebug(KDLS) << "fullConfig:" << fullConfig.toString();

Expand Down

0 comments on commit 97ecca2

Please sign in to comment.