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

4.5:add winch options #5472

Merged
merged 1 commit into from
Jan 31, 2024
Merged

Conversation

Hwurzburg
Copy link
Contributor

No description provided.

@Hwurzburg Hwurzburg requested a review from rmackay9 October 9, 2023 20:05
=== ======
Bit Option
=== ======
0 Spin freely on startup allows the line to be pulled out after startup. This is the current default but is actually quite annoying because we need to remember to disengage the clutch before takeoff or the package will likely be left on the ground.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can shorten this one to just, "Spin freely on startup allows the line to be pulled out after startup"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will do

@rmackay9
Copy link
Contributor

Great, thanks very much for this. I've suggested one change but after that it's good to merge.

@Hwurzburg
Copy link
Contributor Author

its a 4.5 merge....and is already linked in upcoming...all marked 4.5 wiki are there

@rmackay9
Copy link
Contributor

rmackay9 commented Oct 11, 2023

ok, so I would have expected that I could find the new lidar wiki page in the upcoming section. https://ardupilot.org/copter/docs/common-master-features.html... but I guess you're thinking we should not have a page in the wiki at all until we've started 4.5 beta testing? That seems too late to me actually.

@Hwurzburg
Copy link
Contributor Author

as I said its already been linked
Capture

@rmackay9 rmackay9 mentioned this pull request Oct 11, 2023
@Hwurzburg Hwurzburg merged commit f7f60ef into ArduPilot:master Jan 31, 2024
4 checks passed
@Hwurzburg Hwurzburg deleted the 4.5/Winch_options branch January 31, 2024 23:21
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.

2 participants