ArduPilot / ArduPilot/ardupilot

Winch: add support for WINCH_DELIVER action

Open
#23,976 3 comments 0 reactions 0 assignees View on GitHub
Enhancement
Dominant language
C++
Stars
15.9k
Forks
21.4k
Avg merge
3d 17h
Merged PRs (30d)
119

Description

[MAVLink's DO_WINCH command](https://github.com/ArduPilot/mavlink/blob/master/message_definitions/v1.0/common.xml) defines action "4" as "WINCH_DELIVER" meaning that winch is lowered until the package reaches the ground.

We could add support for this (perhaps using [Payload Place](https://ardupilot.org/copter/docs/mission-command-list.html#payload-place)) when using the [Daiwa winch](https://ardupilot.org/copter/docs/common-daiwa-winch.html) based on the reported tension suddenly reducing. The [Daiwa winch driver is here](https://github.com/ArduPilot/ardupilot/blob/master/libraries/AP_Winch/AP_Winch_Daiwa.h).

This enhancement request comes from [this discussion](https://github.com/ArduPilot/ardupilot/issues/23976).

Contributor guide

Open the contributing guide

Research direction

Start with the MAVLink DO_WINCH definition and the Daiwa driver in libraries/AP_Winch/AP_Winch_Daiwa.h, then compare the Payload Place behavior in the linked mission-command documentation. Determine how WINCH_DELIVER should lower the package and use the reported sudden tension reduction to detect ground contact. Done means the Daiwa winch supports action 4 with behavior consistent with the requested delivery semantics.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
embedded-iot, robotics
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.