-
Notifications
You must be signed in to change notification settings - Fork 160
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
Speed values and Q matrix #50
Comments
I’ve also been working on fixing the speed reading and this is what I came up with (in C++):
void GPSAccKalmanFilter::rebuildQ(double dtUpdate, double accDev) {
double velDev = accDev * dtUpdate * 100;
double posDev = velDev * dtUpdate * 100;
double covDev = velDev * posDev;
double posSig = posDev * posDev;
double velSig = velDev * velDev;
m_kf.Q.setData(16,
posSig, 0.0, covDev, 0.0,
0.0, posSig, 0.0, covDev,
covDev, 0.0, velSig, 0.0,
0.0, covDev, 0.0, velSig);
}
… On Sep 2, 2019, at 8:52 AM, Lucas Thom Ramos ***@***.***> wrote:
Can someone tell me why rebuildQ is in function of m_predictCount since the accDev does not change over time?
Also, do you have some clue on why the speed readings are so wrong?
Thank you
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub <#50?email_source=notifications&email_token=AACQKTTZPAUTLBIF3OG6AP3QHUZEHA5CNFSM4IS6XBZ2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HI2DP5Q>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AACQKTWOYIXQQ3J6HMSNLALQHUZEHANCNFSM4IS6XBZQ>.
|
@Pstoppani I'm also using a similar approach:
I'm working on android btw, there was also a wrong size of B matrix and u vector. They should be (4x2) and (2x1) respectively. Btw, now it seems the vectors (east, north) from accelerometer are not on the same orientation as the gps speed vectors. |
I think it's also a mistake to use the orientation from the gps on update cause we have this data more accurate from magnetometer. |
Agreed. I did the same thing.
… On Sep 2, 2019, at 12:41 PM, Lucas Thom Ramos ***@***.***> wrote:
I think it's also a mistake to use the orientation from the gps on update cause we have this data more accurate from magnetometer.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <#50?email_source=notifications&email_token=AACQKTWJLDGA34GMP24N2DDQHVT6TA5CNFSM4IS6XBZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5WOWBI#issuecomment-527231749>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AACQKTWB4VX3PFPQACY6VU3QHVT6TANCNFSM4IS6XBZQ>.
|
@thxmxx @Pstoppani can you elaborate on what you changed for gps orientation? |
Sorry, I misunderstood the previous comment. I think it is fine to use the GPS orientation on update since the accel “predicts” are from the IMU at a higher rate than GPS. The Kalman filter combines them properly and I think it is best to use both GPS and IMU orientation.
… On Sep 16, 2019, at 8:21 AM, Martin Guillon ***@***.***> wrote:
@thxmxx <https://github.com/thxmxx> @Pstoppani <https://github.com/Pstoppani> can you elaborate on what you changed for gps orientation?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <#50?email_source=notifications&email_token=AACQKTVVLOADCNVCZDTAKYTQJ6QAHA5CNFSM4IS6XBZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6ZQLVI#issuecomment-531826133>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AACQKTWT52ORHCU7SLSAXE3QJ6QAHANCNFSM4IS6XBZQ>.
|
@Pstoppani thanks! Will test it on my side. I ported the code to JS as i use it within Nativescript cross platform environment. |
@farfromrefug Instead of using |
@thxmxx thanks a lot! will use that too. |
@farfromrefug you should also use this #51 |
I'm getting the error of +-5Km/h. But I'm still working on that, since I'm using GPS speed on updates and the predict results are still bad. |
@thxmxx Thanks for the infos! My question is this. When updating the Kalman filter with the GPS location, what we are doing is using GPS speed as a reference for the Kalman Filter to "predict" it afterwards based on Linear ACceleration, right? I am not that familiar with the Kalman Filter What i am seeing when outside is that the GPS speed pretty quickly goes to 0 when i stop walking. But the Filtered speed does not want to go to 0. It just slowly goes down. EDIT: found out that article. And i think it is the inspiration for that repo |
Can someone tell me why rebuildQ is in function of m_predictCount since the accDev does not change over time?
Also, do you have some clue on why the speed readings are so wrong?
Thank you
The text was updated successfully, but these errors were encountered: