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

[modules] wing-rotation controller with servo #3143

Merged
merged 5 commits into from
Nov 9, 2023

Conversation

dewagter
Copy link
Member

No description provided.

@dewagter dewagter mentioned this pull request Oct 18, 2023
64 tasks
@dewagter
Copy link
Member Author

How do you enable ADC_5 in the test architecture?

@fvantienen
Copy link
Member

You should make ADC_CHANNEL_WING_ROTATION_POSITION a configure. Then you can do something similar to:

    <configure name="ADC_CHANNEL_WING_ROTATION_POSITION" default="ADC_5" case="lower|upper"/>
    <define name="ADC_CHANNEL_WING_ROTATION_POSITION" value="$(ADC_CHANNEL_WING_ROTATION_POSITION_UPPER)"/>
    <define name="USE_$(ADC_CHANNEL_WING_ROTATION_POSITION_UPPER)"/>

@fvantienen
Copy link
Member

That way the ADC is enabled by itself instead of doing in the airframe

@fvantienen
Copy link
Member

Btw the naming of these defines etc. could be a lot better. Usually it is best to start with the module name, such that they are all grouped together

@dewagter dewagter force-pushed the wing_rotation_controller branch from 351d4b4 to 7aa50f0 Compare October 19, 2023 14:38
@dewagter dewagter requested a review from fvantienen October 19, 2023 14:54
@dewagter
Copy link
Member Author

@fvantienen update complete

@dewagter dewagter dismissed fvantienen’s stale review October 19, 2023 19:14

solved comments

@dewagter dewagter marked this pull request as draft October 23, 2023 14:45
@dewagter dewagter force-pushed the wing_rotation_controller branch 2 times, most recently from 021b1c6 to d38bc88 Compare November 6, 2023 12:21
@gautierhattenberger
Copy link
Member

is is still a draft ?

@dewagter
Copy link
Member Author

dewagter commented Nov 7, 2023 via email

update

Cleanup

rename defines

renames

cleanup airframe

renames

remove airspeed

Send wing position feedback via ABI (like on the 25kg in AP mode)
@dewagter dewagter force-pushed the wing_rotation_controller branch from d38bc88 to 6003201 Compare November 7, 2023 21:00
@dewagter dewagter marked this pull request as ready for review November 7, 2023 21:14
@dewagter
Copy link
Member Author

dewagter commented Nov 7, 2023

Cleanup complete.

@dewagter dewagter merged commit 8d40909 into master Nov 9, 2023
@dewagter dewagter deleted the wing_rotation_controller branch November 9, 2023 14:10
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