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

Updated librealsense2 to version 2.38.1 #946

Merged
merged 5 commits into from
Oct 6, 2020

Conversation

cansik
Copy link
Member

@cansik cansik commented Sep 18, 2020

This PR updates the librealsense2 library to version 2.38.1 which supports newer devices like the RealSense L515 (LIDAR). There were not that many changes necessary, mainly the libtm which has been replaced with tm2.

I could just test the library on MacOS but there the new devices are recognized and an output image could be captured.

I was not sure if it's necessary to update the number 1.5.4 to 1.5.5, but I did it. Maybe you would like to change it (back or to another version).

@saudet
Copy link
Member

saudet commented Sep 19, 2020

Thanks for the update! Use 1.5.5-SNAPSHOT for the version until it gets released.
Before we can merge though we'll need the builds to pass. They are currently on Linux and Windows:
https://travis-ci.org/github/bytedeco/javacpp-presets/builds/728431118
https://ci.appveyor.com/project/Bytedeco/javacpp-presets/builds/35285006

@saudet
Copy link
Member

saudet commented Sep 22, 2020

On Linux, we can get the video.h, etc files as with this patch used for librealsense:
https://github.com/bytedeco/javacpp-presets/blob/master/librealsense/librealsense.patch
For Windows, we'll need to skip the missing symbols, which are undefined anyway, so we don't lose anything.

@cansik
Copy link
Member Author

cansik commented Sep 22, 2020

I will have a look at it tomorrow! Thx for the investigation.

@saudet
Copy link
Member

saudet commented Sep 22, 2020 via email

@saudet saudet merged commit 55e0e0f into bytedeco:master Oct 6, 2020
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

2 participants