Skip to content

AP_Scripting: add SITL RealFlight gimbal driver#27752

Open
robertlong13 wants to merge 1 commit into
ArduPilot:masterfrom
robertlong13:pr/sitl-realflight-gimbal
Open

AP_Scripting: add SITL RealFlight gimbal driver#27752
robertlong13 wants to merge 1 commit into
ArduPilot:masterfrom
robertlong13:pr/sitl-realflight-gimbal

Conversation

@robertlong13

@robertlong13 robertlong13 commented Aug 5, 2024

Copy link
Copy Markdown
Contributor

Useful for testing this PR: ArduPilot/MissionPlanner#3386

Goes along with this SITL Models PR: ArduPilot/SITL_Models#140

@Georacer

Georacer commented Aug 5, 2024

Copy link
Copy Markdown
Contributor

Perhaps it's a silly question, but: What makes this driver targeted to RealFlight? Isn't it applicable to any PWM gimbal?

@robertlong13

robertlong13 commented Aug 5, 2024

Copy link
Copy Markdown
Contributor Author

Perhaps it's a silly question, but: What makes this driver targeted to RealFlight? Isn't it applicable to any PWM gimbal?

No, you're right, it would be applicable to any PWM gimbal that stabilizes roll pitch and yaw internally, takes PWM commands for earth-frame pitch angle command and body-frame yaw angle command, and has a servo channel for zooming the camera (though that last part is optional). If there are any of that kind of gimbal still on the market (I feel like they all dried up in 2017) then this would be a nice backend for that.

The only reason this has RealFlight written all over it is that I made it for RealFlight and it works well with it.

@Georacer

Georacer commented Aug 6, 2024

Copy link
Copy Markdown
Contributor

Then if it's not too much trouble, I'd prefer seeing this be turned into such a generic PWM driver. Especially since it's placed in the drivers folder.
I realize it might need a bit more scrutiny in that case.

Of course, staying a RealFlight script (and perhaps moving it to examples) is also fine by me.

@robertlong13

robertlong13 commented Aug 11, 2024

Copy link
Copy Markdown
Contributor Author

Then if it's not too much trouble, I'd prefer seeing this be turned into such a generic PWM driver. Especially since it's placed in the drivers folder. I realize it might need a bit more scrutiny in that case.

If someone has one they want to flight test for me, then I'd happily make it generic.

I can't even find one to buy that isn't STorM32 based, which wouldn't need this driver.

@robertlong13 robertlong13 force-pushed the pr/sitl-realflight-gimbal branch from f2cb2c6 to 79d258b Compare January 5, 2025 21:30
@isso11

isso11 commented Aug 6, 2025

Copy link
Copy Markdown

Then if it's not too much trouble, I'd prefer seeing this be turned into such a generic PWM driver. Especially since it's placed in the drivers folder. I realize it might need a bit more scrutiny in that case.

If someone has one they want to flight test for me, then I'd happily make it generic.

I can't even find one to buy that isn't STorM32 based, which wouldn't need this driver.

Hi @robertlong13
I can test this on a real hardware. I have some gimbals at my university and can test if you want.

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.

3 participants