openpilot_comma

15868 Commits

111 Branches

84 Tags

5.8 GiB

Tag: Branch: Tree: 0766a94797

Commit Graph

Author SHA1 Message Date

Author	SHA1	Message	Date
DevTekVE	c236f472a9	update ISO_LATERAL_ACCEL import + VM changes (#35865 ) * refactor: move lateral methods from init to lateral.py (#2594) * Extracting lateral methods to lateral.py * cleaning * more cleaning * more cleaning * Making sure it remains where it should * Leave rate_limit where it belongs * Moving things to `car/controls/` * Moving rate limit to get a taste of the changes * clean * copy verbatim * clean up * more * now we can format --------- Co-authored-by: Shane Smiskol <shane@smiskol.com> * Merge branch 'master' into move-common-vm-methods. dunno what happend with ci * now we need to move this import * bump opendbc --------- Co-authored-by: Shane Smiskol <shane@smiskol.com> Co-authored-by: Adeeb Shihadeh <adeebshihadeh@gmail.com>	3 months ago
Shane Smiskol	c35494c19f	Check for excessive lateral acceleration (#35921 ) * here? * nah card shouldn't become bloated * better * import * actually selfdrived is probably best since it already manages alerts card is car interfacing, controlsd is for calculating control input, selfdrived is rest * consistent name * add to params * ai * maybe better? * shorter * build out lockout * do * check active * descriptive * this is a terrible experience just to get lat accel * just pass sm * not iso * type * rm * math * use calibrated roll * fix * fix borkenness * cmt * compare some methods * rolling window * 1 and 2 are the same * rm it * stuff * plot * plot kf * generic implementation * adjust limits * fix from merge * clean up * revert filter to master * and here * and * run_process_on_route imps * clean up * why not * extrapolate * this doesn't generically work for angle/curvature cars Revert "extrapolate" This reverts commit `556f0c3a92`. * cmt * move * rm debug rm debug and * others use helpers * two counters might be too much to return * turn into class * clean up * cmt * kinda obvious * impossible for this not to be true, but make it explicit * clean up	3 months ago

DevTekVE

c236f472a9

update ISO_LATERAL_ACCEL import + VM changes (#35865 )

* refactor: move lateral methods from init to lateral.py (#2594)

* Extracting lateral methods to lateral.py

* cleaning

* more cleaning

* more cleaning

* Making sure it remains where it should

* Leave rate_limit where it belongs

* Moving things to `car/controls/`

* Moving rate limit to get a taste of the changes

* clean

* copy verbatim

* clean up

* more

* now we can format

---------

Co-authored-by: Shane Smiskol <shane@smiskol.com>

* Merge branch 'master' into move-common-vm-methods. dunno what happend with ci

* now we need to move this import

* bump opendbc

---------

Co-authored-by: Shane Smiskol <shane@smiskol.com>
Co-authored-by: Adeeb Shihadeh <adeebshihadeh@gmail.com>

3 months ago

Shane Smiskol

c35494c19f

Check for excessive lateral acceleration (#35921 )

* here?

* nah card shouldn't become bloated

* better

* import

* actually selfdrived is probably best since it already manages alerts

card is car interfacing, controlsd is for calculating control input, selfdrived is rest

* consistent name

* add to params

* ai

* maybe better?

* shorter

* build out lockout

* do

* check active

* descriptive

* this is a terrible experience just to get lat accel

* just pass sm

* not iso

* type

* rm

* math

* use calibrated roll

* fix

* fix borkenness

* cmt

* compare some methods

* rolling window

* 1 and 2 are the same

* rm it

* stuff

* plot

* plot kf

* generic implementation

* adjust limits

* fix from merge

* clean up

* revert filter to master

* and here

* and

* run_process_on_route imps

* clean up

* why not

* extrapolate

* this doesn't generically work for angle/curvature cars

Revert "extrapolate"

This reverts commit 556f0c3a92.

* cmt

* move

* rm debug

rm debug

and

* others use helpers

* two counters might be too much to return

* turn into class

* clean up

* cmt

* kinda obvious

* impossible for this not to be true, but make it explicit

* clean up

3 months ago

2 Commits (0766a94797b2bd4558e8b466e94fd168ca407b12)