-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
android - add presets support, enable camera stream configuration #3771
android - add presets support, enable camera stream configuration #3771
Conversation
src/android/jni/advanced_mode.cpp
Outdated
Java_com_intel_realsense_librealsense_Device_nSerializePresetToJson(JNIEnv *env, jclass type, | ||
jlong handle) { | ||
rs2_error* e = NULL; | ||
rs2_raw_data_buffer* raw_data_buffer = rs2_serialize_json( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The raw_data_buffer should probably be wrapped in RAII in order jByteArray will throw.
Check the implementation for std::vector<uint8_t> send_and_receive_raw_data(con
in rs_device.hpp
src/android/usb_host/usb_pipe.cpp
Outdated
for(int i = 0; i < response->actual_length; i++) | ||
buff += ", " + std::to_string(((uint8_t*)response->buffer)[i]); | ||
LOG_DEBUG("interrupt_request: " << buff.c_str()); | ||
if(response->actual_length == 6){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Replace all the hard-coded codes with enum for maintainability
long[] streamProfilesHandles = nGetStreamProfiles(mHandle); | ||
List<StreamProfile> rv = new ArrayList<>(); | ||
for(long h : streamProfilesHandles){ | ||
rv.add(new VideoStreamProfile(h));// TODO check if extendable |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Put provision for motion sensor
src/android/usb_host/usb_pipe.cpp
Outdated
0x01, //CLEAR_FEATURE | ||
0, _endpoint.get_endpoint_address(), NULL, 0, 10) == UVC_SUCCESS; | ||
0x01, //CLEAR_FEATURE | ||
0, _endpoint.get_endpoint_address(), NULL, 0, 10) == UVC_SUCCESS; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add comment for 10
This PR enables depth presets via Java Android wrapper.
It also contains new features to the Android camera app:
Tracked on: DSO-12447