forked from commaai/openpilot
-
Notifications
You must be signed in to change notification settings - Fork 55
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
first version with a lot TODOs #499
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Looks good |
arne182
added a commit
that referenced
this pull request
Dec 8, 2019
* updates to long_mpc * 066 updates (#208) * add custom error messages to arne182.capnp * add explanations for some parameters in op_params, add parameter to specify custom following distance updates to dynamic_follow: - fix v_lead that should have been a_lead - made relative velocity mod more aggressive * make customTR a bit safer * perhaps this will work * remove travis detected message * start updating op_params, fix issue with lane_hugging mod where it wouldn't work with right hugging * revert custom alert message * testing new op_edit * testing new op_edit * testing new op_edit * testing new op_edit * testing new op_edit * testing new op_edit * testing new op_edit * testing new op_edit * testing new op_edit * add description! * add description! * testing op_edit * testing op_edit * testing op_edit * testing op_edit * testing op_edit * testing op_edit * testing op_edit * testing op_edit * testing op_edit * testing op_edit * testing op_edit * finish updating opEdit * final update * tuned dynamic follow relative velocity values to be slightly more aggressive at high rel vels fix error in get_cost function causing it to always return 0.1 above 5 mph * updates to long control, testing without dynamic gas * testing customTR * testing old cost function * revert to 066-clean * revert to 066-clean * use clip instead * updates to long_mpc * revert TR values * updates to long_mpc * revert new costs to how they were in 05 * refactor, remove unneeded variable * Create __init__.py * At least 2m needed to keep stopping for traffic lights * Refine one-way logic Now both roads need to be a one way and it will only choose it if there is one way in the right direction and the other is not. * fix for right hand turns not slowing down. * remove integrated value on release and not press of brake/gas * new lane hugging mod, seems to work a lot better! tuning for corolla steering, less seems to be better * time wasn't imported * tiny bit of refactor * Try the reset for pid after gas pedal is * treat brake pressed or gas pressed as LongCtrlState.off * I also want events * Fix hex value * add possibility for arne182 events * fix * fix arne capnp * return ret_arne182 * Try and combine events together. This could probably not work * think this will be better, we want to modify as little of comma's code as possible * test * test if messages from arne.capnp will be accepted by AM * test combined events * this should work * final changes to make custom alerts work only for toyotas for now * Stop data_send from crashing because of unknown events * Pass in sendevents to state_transition * try prepend * Append lists together * Thank goodness for syntax highlighting * fix right hand turns The negative value is already in steering angle which causes tan to be negative so we do not need an extra minus there. * Completely seperate events_arne182 and events * Add arne_ret to all interfaces (#497) * add ret_arne to honda.interface * add ret_arne to chrysler.interface * add ret_arne to rest of interfaces * first version (#499) * Add exception for gas and brake press for planner This could be the cause of the steering braking after manual gas and brake input * Try remove red screen and loud beep at low speed reverseGear Engagement * Add custom alert without chimeWarningRepeat * Use new custom reverse Warning without chime * fix create_event_arne * Update arne182.capnp * Fix
rav4kumar
pushed a commit
that referenced
this pull request
Apr 20, 2020
884afa0 Safety Test Refactor: Chrysler and Volkswagen PQ (#508) d77b72d Safety Test Refactor: Nissan (#510) 4c7755c Match Panda DFU entry fix in "make recover" process (#509) 0336f62 Pedal gas pressed safety limits (#507) 715b1a1 Hyundai-Kia-Genesis (HKG) (#503) 6f105e8 Safety Test Refactor: Subaru (#502) 57cc954 Safety Test Refactor: GM (#504) dd01c3b Safety Test Refactor: Hyundai (#505) 592c2c8 add support to can_unique.py for Cabana CSV format. (#506) ccf13b7 No more infinite while loops in CAN init (#499) 6c442b4 Safety Test Refactor: Volkswagen MQB (#493) f07a6ee panda recover should go through bootstub first (#498) 8cc3a35 remove cadillac (#496) 62e4d3c Chrysler: fix missing button signal on TX (#490) abce8f3 Safety Test Refactor: Toyota + support code (#491) 500370a Make sure relay faults make it to the health packet (#492) bc90b60 toyota: use universal gas pressed bit (#488) 74d10cc Fixed possible race condition (#487) a05361e cleanup safety_replay dockerfile (#486) fe73dcc Openpilot-tools is deprecated (#484) da8e00f TX message guaranteed delivery (#421) d8f6184 Add ISO number for longitudinal limits flag comment 6a60b78 touch ups 2ce6536 comments on unsafe flags d880134 remove from there as well 055ea07 remove that unsafe flag since it isn't implemented and it's unclear how to 4e98bbe Apply unsafe allow gas mode to all cars. (#480) 0c2c149 Fixing libusb busy error (#174) 753c42c Update Board Mac SDK Install script to work on clean mac (#146) b9a9ea3 Unsafe gas disengage mods, fix test compile warning (#481) 08ef92d Safety model for Volkswagen PQ35/PQ46/NMS (#474) 51e0a55 Support code for unsafe mode unit tests (#478) 5325b62 current_safety_mode 7908b72 update updating unsafe mode 98503e8 disable stock honda AEB in unsafe mode (#477) 01b2ccb one more 9a30265 weak steering while not engaged 577f10b added options for unsafe mode 83cf7bf update comment 4556e74 enable unsafe mode, toggle for use by forks that so choose de89fcd Nissan leaf (#473) git-subtree-dir: panda git-subtree-split: 884afa0
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.