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

IMU Noise model #74

Open
RyodoTanaka opened this issue Dec 4, 2023 · 6 comments
Open

IMU Noise model #74

RyodoTanaka opened this issue Dec 4, 2023 · 6 comments
Assignees
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@RyodoTanaka
Copy link
Member

We do not set any noise model for IMU.
The usual IMU they have several noise model.
I think following noise model is needed mainly.

  1. Gussian Noise
  2. Draft Noise
    This noise could be represented by using time scale.
  3. Temperature Noise

If anyone have any other idea, it is very welcome to write them on here.

@RyodoTanaka RyodoTanaka added enhancement New feature or request help wanted Extra attention is needed labels Dec 4, 2023
@qoopen0815
Copy link
Member

I'm interested in implementing the proposed noise model, but I'm a little unsure where to stick it in my code. Could you please throw a hint as to which part of the codebase is needed?

@Autumn60
Copy link
Contributor

@qoopen0815

Thank you for your interest.
Currently, codes for the noise model are located in the following directory

https://github.com/Field-Robotics-Japan/UnitySensors/tree/master/Assets/UnitySensors/Runtime/Scripts/Utils/Noise

Also, the namespace would be UnitySensors.Utils.Noise.

@qoopen0815
Copy link
Member

@Autumn60
Thanks, I found the code!

Quick question:
Any reason the noise model, which seems ready to roll, hasn't been integrated into the IMU yet? Is there a specific reason or is the implementation approach still up in the air?

@Autumn60
Copy link
Contributor

That's because the implemention approach was not set in stone.

I would like to have a framework that allows us to combine multiple noise models in the future.
(e.g., Gaussian noise + drift noise, Gaussian noise + temperature noise, etc.)
Given this, I believe it is necessary to have a common interface for all noise model classes, like PointCloud data.

@Autumn60
Copy link
Contributor

Autumn60 commented Feb 9, 2024

https://github.com/ethz-asl/kalibr/wiki/IMU-Noise-Model

@RyodoTanaka
Copy link
Member Author

@Autumn60
The ethz-asl's link you've found looks very interesting !
It seems we also could be possible to implement the noise model 😎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

When branches are created from issues, their pull requests are automatically linked.

3 participants