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.

Sign up for GitHub

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

Jump to bottom

Fix Python tools - file name consistency - compile master without warnings - airframe fix #3388

Merged
merged 15 commits into from
Oct 4, 2024
Merged

Fix Python tools - file name consistency - compile master without warnings - airframe fix #3388

merged 15 commits into from
Oct 4, 2024

Conversation

dewagter
Copy link
Member

@dewagter dewagter commented Oct 3, 2024 •
edited
Loading

  • Moving defines to *.h files requires compiler defines. airframe.h in driver is not working.
  • More flightplan renames to rotwing_
  • update IMU locations
  • Fix python tools
  • redundant gih_param
  • removed unfixable bebop
  • typo 13 .0
  • fix rotwing_state_V2
  • ins_ext_vision only works in AP
  • rotwing_viewer.py update got lost

@dewagter dewagter marked this pull request as draft October 3, 2024 12:56
@dewagter dewagter self-assigned this Oct 3, 2024
@dewagter dewagter marked this pull request as ready for review October 3, 2024 14:25
@dewagter dewagter changed the title Compile Master Without Warnings Fix Python tools - file name consistency - compile master without warnings - airframe fix Oct 3, 2024
@dewagter dewagter mentioned this pull request Oct 3, 2024
Closed
33 tasks
EwoudSmeur
EwoudSmeur previously approved these changes Oct 3, 2024
View reviewed changes
Copy link
Member

@EwoudSmeur EwoudSmeur left a comment

Choose a reason for hiding this comment

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

LGTM, though the ekf2 airframe changes can go to rotwing_common

conf/airframes/tudelft/rotwing_v3b.xml Outdated Show resolved Hide resolved
@dewagter dewagter mentioned this pull request Oct 3, 2024
Open
@dewagter dewagter added the bug label Oct 3, 2024
@dewagter
Copy link
Member Author

dewagter commented Oct 3, 2024

Zero error/warning on make test_all_confs + simulated several airframes (but not flight tests)

@gautierhattenberger gautierhattenberger merged commit d7c50a2 into master Oct 4, 2024
1 check passed
@gautierhattenberger gautierhattenberger deleted the final_tweaks branch October 4, 2024 06:51
NoahWe pushed a commit to tudelft/paparazzi that referenced this pull request Oct 14, 2024
@dewagter @NoahWe
Fix Python tools - file name consistency - compile master without war…
e975127 
…nings - airframe fix (paparazzi#3388)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gautierhattenberger gautierhattenberger gautierhattenberger approved these changes

@EwoudSmeur EwoudSmeur Awaiting requested review from EwoudSmeur

@Fabien-B Fabien-B Awaiting requested review from Fabien-B

Assignees

@dewagter dewagter

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dewagter @gautierhattenberger @EwoudSmeur