-
Notifications
You must be signed in to change notification settings - Fork 17.7k
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
Make blended backend a real backend #26919
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.
Very cool changes!
ef37c2f
to
6d4cf64
Compare
we will do some SpringValley flight testing |
6d4cf64
to
5c6c678
Compare
5c6c678
to
23ef206
Compare
15f7df7
to
535744d
Compare
535744d
to
e2ac28f
Compare
Flight testing at SV went well today. |
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.
how does this interact with EKF3 affinity?
@@ -606,7 +604,7 @@ class AP_GPS | |||
protected: | |||
|
|||
// configuration parameters | |||
Params params[GPS_MAX_RECEIVERS]; | |||
Params params[GPS_MAX_INSTANCES]; |
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.
can we remove GPS_MAX_RECEIVERS now?
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.
Not yet. There are all sorts of interesting decisions to be made revolving around it.
Many of them will end up turning into a :
-style iteration across the instances array.
I'd like to understand how this interacts with affinity |
8b53432
to
de1a9bc
Compare
Is this planned for 4.6? |
61f5699
to
06ef30a
Compare
06ef30a
to
c695eae
Compare
I've flown a real copter with blended a bunch now - even with different GPS types it copes. I've added an autotest that tests blended/affinity before and after. The logs on master show that when affinity is in play the third core will use the blended GPS. |
I tested this in SITL a while back using some of the SIM* params to muck with the GPS health of GPS 1 and 2 and could not get the vehicle to have any serious issues with blended. I was wanting triple-GPS support, and was interested in this. The new test is nice to ensure GPS errors get blended out. |
Tested in SITL.