move-fast / openpilot

openpilot is an open source driver assistance system. openpilot performs the functions of Automated Lane Centering and Adaptive Cruise Control for over 85 supported car makes and models.
https://comma.ai/
MIT License
21 stars 3 forks source link

Update vision_turn_controller.py #6

Closed oppilot closed 3 years ago

oppilot commented 3 years ago

Add max V 60mph to automatically comment out vision braking on the freeway. Add a parameter that edits all values in this file call overall_V in percentage, 10.0 is 10% more aggressive turns. Because below 60mph vision braking by about 2mph too much and above 60mph, occasional phantom braking and 5mph too much braking regularly.

alfhern commented 3 years ago

Thanks for submitting your idea. The issue is that different cars have different turn performance based on torque limitations , etc... Therefore what might seem too conservative to you might be too aggressive for others. We are therefore trying to keep it as generic as possible until comma brings in end to end longitudinal. I would suggest you fork it and fine tune it for your use case. Cheers.