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

Pr gz sim tiltrotor #24028

Open
wants to merge 2 commits into
base: pr-add-quadtailsitter
Choose a base branch
from
Open

Conversation

Perrrewi
Copy link

@Perrrewi Perrrewi commented Nov 22, 2024

Solved Problem

Moving the tiltrotor from gazebo classic to gz sim as gazebo garden will be depreciated by the end of nov

Fixes #23602

Solution

Adding an airframe and fixed the bug in gz-bridge. PX4 sends an output value from 0 to 1000 of the servo and the GZbridge normalized it to be within +-1. The problem was that gazebo needs a servo input of angles in radian which meant the currently servos only works in the range +-1 rad.

Test coverage

Tested with QGC

Context

Test: Toggling the actuator testing slider with max output value 1000 while gazebo angle limitations were set to 90 deg

Before updating GZMixingInterfaceServo.cpp:
Screenshot from 2024-11-27 16-42-11
Screenshot from 2024-11-27 17-09-25

After updating GZMixingInterfaceServo.cpp:
Screenshot from 2024-11-27 16-41-02

Screenshot from 2024-11-27 17-07-45

@Perrrewi Perrrewi self-assigned this Nov 22, 2024
@Perrrewi Perrrewi mentioned this pull request Nov 22, 2024
30 tasks
Copy link

github-actions bot commented Nov 22, 2024

FLASH Analysis

px4_fmu-v5x
    FILE SIZE        VM SIZE    
--------------  -------------- 
+0.0%    +176  +0.0%    +176    .text
  +0.1%    +172  +0.1%    +172    ROMFS/nsh_romfsimg.c
  +0.0%      +4  +0.0%      +4    [section .text]
-0.8%    -176  [ = ]       0    [Unmapped]
[ = ]       0  +0.0%    +176    TOTAL

px4_fmu-v6x
    FILE SIZE        VM SIZE    
--------------  -------------- 
+0.0%    +144  +0.0%    +144    .text
  +0.1%    +136  +0.1%    +136    ROMFS/nsh_romfsimg.c
  +0.0%      +8  +0.0%      +8    [section .text]
-0.2%    -144  [ = ]       0    [Unmapped]
[ = ]       0  +0.0%    +144    TOTAL

Updated: 2024-11-29T09:15:17

@Perrrewi Perrrewi changed the title Pr gz sim tiltrotor Draft: Pr gz sim tiltrotor Nov 26, 2024
@Perrrewi Perrrewi changed the title Draft: Pr gz sim tiltrotor Pr gz sim tiltrotor Nov 27, 2024
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

Successfully merging this pull request may close these issues.

1 participant