-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Mission Computer and payload computer support #2053
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please rebase on master and resolve conflicts.
PPRZLINK should also be updated (merge of copilot related messages).
@@ -142,7 +142,7 @@ | |||
|
|||
// AUX1 | |||
#if USE_ADC_2 | |||
#define AD1_2_CHANNEL ADC_CHANNEL_IN2 | |||
#define AD1_2_CHANNEL ADC_CHANNEL_IN3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
already fixed in master
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this came from git merge master
- should I rebase or merge-rebase again?
struct UtmCoor_f utm; | ||
utm.zone = nav_utm_zone0; | ||
utm_of_lla_f(&utm, &lla); | ||
//nav_move_waypoint(wp_id, utm.east, utm.north, utm.alt); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please uncomment or remove
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
struct CameraSnapshot cam_snapshot; | ||
struct CopilotStatus copilot_status; | ||
|
||
PPRZ_MUTEX(copilot_cam_snapshot_mtx); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are you using mutex ? There is no threads in this module.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There might be threads in the future - so just added protection and good practice. I would rather have an unused mutex than forget to add it later.
Tested on Bebop and Bebop 2. v4.0.4 Min version set to 3.3 no motor swap issue "fix" unwanted behaviour.
Fixed - now there are no more conflicts |
@gautierhattenberger are we good to merge this? |
Adding functionality for forwarding messages from the mission and payload computers to GCS and vice-versa. Related to paparazzi/pprzlink#51
Also adds copilot module and adds minor changes to the way messages are parsed in the autopilot.
More details and the description of the system is here: https://wiki.paparazziuav.org/wiki/Mission_computer