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

Radard: just get relative speed from model #27493

Merged
merged 12 commits into from
Jun 2, 2023
Merged

Conversation

haraschax
Copy link
Contributor

No description provided.

@jyoung8607
Copy link
Collaborator

jyoung8607 commented Mar 8, 2023

Yesterday I did a little bit of city driving and a lot of highway driving on this code. I hadn't driven on vision with the new height predicting model yet.

I was really impressed with how it felt around-town, and in stop-and-go traffic. Apparently the static aLeadTau can work. That's subjective, I haven't looked at the data for the around-town stuff.

On the highway, it finds mostly stable but very long following distances. For this, I did look at the data. At least on the highway, it looks like the model has a pretty substantial negative bias on the lead speed, to the tune of -1 to -2 m/s.

Looking at a long stretch where I followed the same car, the model predicts the lead going slower than the ego vehicle for the entire time period. Obviously this isn't possible. It makes the planner want to slow down, when in reality we're not getting any closer to the lead, hence very long follow distance. The vEgo-to-model-speed compensation actually makes this mildly worse.

@HaraldSchafer, may I ask why the model outputs absolute lead speeds rather than relative? It seems to do a good-enough (albeit noisy) job inferring lead distance, from which relative speed and then acceleration and absolute speed could be derived. We shouldn't even need a fantastically accurate lead speed, we just need to know if the lead is moving toward us or away from us, and a very rough idea of how fast. That's how humans handle it.

3cfdec54aa035f3f|2023-03-07--18-04-43

image

@morrislee
Copy link
Contributor

morrislee commented Apr 1, 2023

After using the new 2023-driving model, the difference between calibrated to non-calibrated is almost hard to tell. City driving is fine, highway causes oscillation behind lead like rubber banding effect. I ran the plotjuggler on non-calibrated data on my tall vehicle, looks like the vlead data is fairly noisy, but the model leadv3 is pretty smooth. Is the vlead data contaminated somehow?

image

@morrislee
Copy link
Contributor

morrislee commented Apr 2, 2023

I have used the scale difference from vego to temporalPose/trans/0 and applied to leadsV3/0/v/0 and came up with more consistent result here. Using stock ACC to follow a lead on cruise control shows a very steady negative bias, maybe about 3 to 5% slower than actual speed on top of the noisy leadsV3

image

@jyoung8607
Copy link
Collaborator

I ran the plotjuggler on non-calibrated data on my tall vehicle, looks like the vlead data is fairly noisy, but the model leadv3 is pretty smooth. Is the vlead data contaminated somehow?

You're probably graphing with just qlogs. They are decimated to save space and bandwidth. With the qlogs, you're seeing radarState at 5Hz and modelV2 at 0.5Hz, and that's what's giving you the appearance of discontinuity. If you upload rlogs, you can see both radarState and modelV2 at 20Hz.

@haraschax haraschax marked this pull request as ready for review June 2, 2023 00:35
@haraschax haraschax changed the title Correct lead speed in radard Radard: just get relative speed from model Jun 2, 2023
@morrislee
Copy link
Contributor

Are we no longer needing to calibrate the lead speed?

@haraschax
Copy link
Contributor Author

This should have the same effect, but is cleaner.

@haraschax haraschax merged commit a446c1f into master Jun 2, 2023
@haraschax haraschax deleted the correct_lead_speed branch June 2, 2023 21:38
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