search

state-of-the-art / Handy3DScanner

Repository for Handy 3D Scanner
Apache License 2.0
71 stars 15 forks source link

[FEATURE]435i internal Gyro Usage #65

Open Calsa512 opened 4 years ago

Calsa512 commented 4 years ago

Is your feature request related to a problem? Please describe.

A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

Describe the solution you'd like

The Realsense 435i has an internal IMU / accell + gyro. Although the phone's gyro AVR system may work ok, it would be nice to take full advantage of this functionality as sometimes your phone can be slightly skewed from the sensor mount throwing off data accuracy. Though I admit most of the 4xx cameras do not have this ability, so this is likely a low priority feature for such an early in development app.

Describe alternatives you've considered

Currently through AVR the phone does get position data.

Additional context

Add any other context or screenshots about the feature request here.

state-of-the-art commented 4 years ago

Hi @Calsa512 ,

You're right - we need such a feature. Unfortunately librealsense still not supporting internal IMU for Android (IntelRealSense/librealsense#4384)... So we need to wait when they will prepare it.

From our side - hopefully by the end of the week we will introduce the ARCore positioning in v0.5.0 (#14) and this will help with rough positioning of the camera to simplify the future PCL pointclouds registration (#13).

state-of-the-art commented 4 years ago

Looks like the guys from realsense added IMU support for version 2.26.0 (at least the logic I added to remove IMU init on adnroid was changed: https://github.com/IntelRealSense/librealsense/commit/f0141ceed#diff-76788c4833e724cad4a926d3a6e6de14L851 ).

Need to check on practice - but right now the main blocker for using version > 2.20.0 is IntelRealSense/librealsense#4215 - bulk transfer is not working well.