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.

Sign up for GitHub

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

Jump to bottom

Simulator does not start with IMU_ACCEL_ #3135

Closed
dewagter opened this issue Oct 6, 2023 · 3 comments
Closed

Simulator does not start with IMU_ACCEL_ #3135

dewagter opened this issue Oct 6, 2023 · 3 comments
Labels
Simulation

Comments

@dewagter
Copy link
Member

dewagter commented Oct 6, 2023

No description provided.

@dewagter dewagter mentioned this issue Oct 6, 2023
Closed
64 tasks
@dewagter dewagter changed the title Simulator does not start with _ACCEL_ Simulator does not start with IMU_ACCEL_ Oct 6, 2023
@gautierhattenberger
Copy link
Member

Do you have more details ??

@dewagter
Copy link
Member Author

  • when the airframe file contains <define name="IMU_ACCEL_CALIB" with 3 sensors, NPS does not align (IMU_ALIGN remains red in GCS). If you remove any of the 3 it aligns and simulates well.
  • e.g. tudelft/conf.xml with airframe e.g. RotatingWingV3d target NPS: the IMU does not align in simulation.

@dewagter
Copy link
Member Author

@fvantienen thinks the max is 3 IMU's but NPS adds one so 4 is too much.

dewagter added a commit that referenced this issue Dec 17, 2023
@dewagter
[imu] simulating 3 IMU's failed since an NPS IMU is added. Increase M…
39d5eb7 
…AX_IMU to 4 (Solves #3135)
@dewagter dewagter mentioned this issue Dec 17, 2023
Merged
dewagter added a commit to tudelft/paparazzi that referenced this issue Dec 17, 2023
@dewagter
[imu] simulating 3 IMU's failed since an NPS IMU is added. Increase M…
adebfee 
…AX_IMU to 4 (Solves paparazzi#3135)
dewagter added a commit to tudelft/paparazzi that referenced this issue Dec 17, 2023
@dewagter
[imu] simulating 3 IMU's failed since an NPS IMU is added. Increase M…
4dbfacf 
…AX_IMU to 4 (Solves paparazzi#3135)
dewagter added a commit to tudelft/paparazzi that referenced this issue Dec 17, 2023
@dewagter
Squashed commit of the following:
6a9e572 
commit 39d5eb7
Author: Christophe De Wagter <dewagter@gmail.com>
Date:   Sun Dec 17 10:37:41 2023 +0100

    [imu] simulating 3 IMU's failed since an NPS IMU is added. Increase MAX_IMU to 4 (Solves paparazzi#3135)
dewagter added a commit to tudelft/paparazzi that referenced this issue Dec 17, 2023
@dewagter
Squashed commit of the following:
dd912ea 
commit 92325c6
Author: Christophe De Wagter <dewagter@gmail.com>
Date:   Wed Dec 13 11:07:33 2023 +0100

    ---- ROTWING ADDED ---- extra airframes and FP

    Read feedback from multiple CAN busses...

    extra commands as test

    Updated 3d and 3e to increase roll stability

    Made some fixes for now

    [modules] Add offset and scaling to UAVCAN airspeed sensor

    Updated airspeed and servo endpoints

    Added v3e to conf again

    Airspeed debugging

    [electrical] Add board voltage

    [energy] Test energy message

    [airspeed] Fix ms45xx pressure compensation on board voltage

    [conf] Fix offset

    Testing airspeed fixes

    [arch] Fix i2c invalidate buffer when reading

    Harmonized rotwing airframes

    [checks] Preflight check actuators

    [checks] PFC actuators

commit 4dbfacf
Author: Christophe De Wagter <dewagter@gmail.com>
Date:   Sun Dec 17 10:37:41 2023 +0100

    [imu] simulating 3 IMU's failed since an NPS IMU is added. Increase MAX_IMU to 4 (Solves paparazzi#3135)
gautierhattenberger pushed a commit that referenced this issue Dec 17, 2023
@dewagter
[imu] simulating 3 IMU's failed since an NPS IMU is added. Increase M…
b040314 
…AX_IMU to 4 (Solves #3135) (#3211)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Simulation
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@gautierhattenberger @dewagter