librealsense icon indicating copy to clipboard operation
librealsense copied to clipboard

android-examples-recording-plus

Open oddityyyy opened this issue 1 year ago • 2 comments

What is the purpose of the change

Solve #9989 #10310 #10007 #8551 #5446,and give a unified and complete solution. Make changes on the basis of the https://github.com/IntelRealSense/librealsense/tree/master/wrappers/android/examples/recording example in the Android sample program, introduce the openCV library, realize pressing the snapshot button, save the aligned color map, pseudo-color depth map, and grayscale depth map locally on the Android device, and use the getData method to save The depth data is a .csv file.

Does this pull request potentially affect existing recording functions?

No. Snapshot and recording are two independent processes and do not affect each other. Users can choose to press the snapshot button to save the frame data at a certain moment to the Android local or press the recording button to record a complete bag file for a certain period of time according to their own needs.

oddityyyy avatar Jun 17 '23 15:06 oddityyyy

Hi @oddityyyy This PR is so big!

  1. please rebase your code base to the current development branch.
  2. too many codes have been changed outside of the Android wrapper folder. for example, csharp/dlib/matlab/nodejs/open3d/opencv/openvino/pcl/pointcloud/python/unrealengine4... some of them have been changed/removed in previous release version. I think you should not modify them.
  3. is it possible that move opencvlibray out of this repo? or add opencv using .gitsubmodule?
  4. both "capture" and "recording" are changed. but in the description of this PR, only one modified example is "recording". do you want to change all of them?
  5. which version of Android Studio are you using?

and one more question, why do saving .jpg files instead of extracting pictures from the recorded .bag file?

Thx.

mengyui avatar Jun 21 '23 09:06 mengyui

Hi @oddityyyy This PR is so big!

  1. please rebase your code base to the current development branch.
  2. too many codes have been changed outside of the Android wrapper folder. for example, csharp/dlib/matlab/nodejs/open3d/opencv/openvino/pcl/pointcloud/python/unrealengine4... some of them have been changed/removed in previous release version. I think you should not modify them.
  3. is it possible that move opencvlibray out of this repo? or add opencv using .gitsubmodule?
  4. both "capture" and "recording" are changed. but in the description of this PR, only one modified example is "recording". do you want to change all of them?
  5. which version of Android Studio are you using?

and one more question, why do saving .jpg files instead of extracting pictures from the recorded .bag file?

Thx.

Small comment about 3, we prefer using ExternalProject_Add on Cmake on top of submodule.

Nir-Az avatar Jun 21 '23 10:06 Nir-Az