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

nMeasurements < nDof; #13

Open
aditdoshi333 opened this issue Sep 14, 2023 · 4 comments
Open

nMeasurements < nDof; #13

aditdoshi333 opened this issue Sep 14, 2023 · 4 comments

Comments

@aditdoshi333
Copy link

Facing this error a lot of times. Can you please explain why it is happening and what scene it can happen?

Please share some light to solve this issue.

@CharlieV5
Copy link

I have encountered this issue too, with the rotating velodyne LiDAR. This often happens in narrow space, maybe . Looking forward to the author's explaining.

@satyajitghana
Copy link

satyajitghana commented Sep 19, 2023

The author has removed code for if (nMeasurements < nDof) which was there in FASTLIO2. due to integration with VoxelMap.

@CharlieV5
Copy link

The author has removed code for if (nMeasurements < nDof) which was there in FASTLIO2. due to integration with VoxelMap.

Could you tell us where the code for if (nMeasurements < nDof) in Fast-LIO2 can be found? I tried but not see it in the latest version.

@satyajitghana
Copy link

You can take a look at this:

Eigen::Matrix<scalar_type, Eigen::Dynamic, Eigen::Dynamic> h_x_cur = Eigen::Matrix<scalar_type, Eigen::Dynamic, Eigen::Dynamic>::Zero(dof_Measurement, n);

Actual Code: https://github.com/hku-mars/FAST_LIO/blob/478d436813b966df94fb91dd759f477f1a246296/include/IKFoM_toolkit/esekfom/esekfom.hpp#L890

You can see lot of lines commented out in PV-LIO which were actual parts of FASTLIO2.

Also another thing to note is that, you can actually disable the voxel based mapping in PV-LIO, which will then rely on the IKFoM based method only using IMU, and if you do that it WILL DRIFT! which it shouldn't, given that FASTLIO2 does not drift.

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

No branches or pull requests

3 participants