Yarp: Move realsense2 YARP devices to a separate standalone repository.

Created on 25 Sep 2020  路  8Comments  路  Source: robotology/yarp

The realsense2 device depends on the realsense2 library that is not available by default in debian/ubuntu.
In order to simplify the installation using binary packages, we should move this device in a separate repository.

CMake Devices Task

Most helpful comment

For small repos, a well written README is definitely the most accessible and maintainable form of documentation.

All 8 comments

I imported the realsense2 device here:
https://github.com/robotology/yarp-device-realsense2

There are probably still a couple of things to fix, but the code is officially there, and the src/devices/realsense2 in YARP is therefore frozen.
Also the documentation in doc/subdevice_realsense.md, should probably be moved in the new repository (cc @aikolina)

I will remove it from YARP as soon as I am sure that it works properly.

I will remove it from YARP as soon as I am sure that it works properly.

Fyi, I tested the new repository and everything seems ok. I tested with a D415 only.

realsense

Thanks @xEnVrE for testing!

Also successfully tested with a D435i (realsense2 device), YARP 3.4.1, RealSense SDK 2.39.0.

Thanks @PeterBowman
I think we are ready to remove the driver from master then...

Also the documentation in doc/subdevice_realsense.md, should probably be moved in the new repository

We should still figure out what to do with this documentation page, since if we move it out of YARP it will no longer be generated, but it doesn't make sense to leave it there, since the device is no longer there.
Perhaps we can convert it to markdown, and adapt it to be the README.md file in the new repository?

For small repos, a well written README is definitely the most accessible and maintainable form of documentation.

Done, updated the PR

Was this page helpful?
0 / 5 - 0 ratings