openpilot icon indicating copy to clipboard operation
openpilot copied to clipboard

Tesla Model 3 / Model Y Car Port

Open lukasloetkolben opened this issue 2 years ago • 1 comments

The best thing about this carport is that only a single bus had to be intercepted. 🎉 No external Panda is required, just a C3X and a harness box.

Checklist Model 3

  • [x] added entry to CAR in selfdrive/car/tesla/values.py
  • [x] test route added to routes.py
  • [x] route with openpilot: b5f2b5b4ef23f4b6/00000008--d88c2210a5
  • [x] route with stock system: b5f2b5b4ef23f4b6/00000009--f3a74d938c
  • [x] car harness used (if comma doesn't sell it, put N/A): N/A

Checklist Model Y

  • [x] added entry to CAR in selfdrive/car/tesla/values.py
  • [x] test route added to routes.py
  • [x] route with openpilot: 35f334fe58c5cdc5/00000002--6261eeae61
  • [x] route with stock system: 35f334fe58c5cdc5/00000001--d8d176b1af
  • [x] car harness used (if comma doesn't sell it, put N/A): N/A

Tesla is in dashcam mode only, so Model 3 and Model Y will not yet appear in CARS.md or release notes.

Additional Routes for the other Fingerprints: Model 3 2023 (RHD): 574d45dbc63e9fd8/00000000--0c80908e8c Model 3 2023 (LHD): 8b67b428fd9fea5c/00000047--d04a4d7f8a Model 3 2021 (LHD): 5df538ba7679f5b7/00000002--ab29e12a18

Demos on Youtube: https://www.youtube.com/watch?v=CwjCp1Gb9d4 https://www.youtube.com/watch?v=1rh0sfZPymE https://www.youtube.com/watch?v=LC0k8hmkBus https://www.youtube.com/watch?v=ljMDHocoxxM

Small fix in Tesla carcontroller.py: Currently in openpilot cancel commands are sent when CS.hands_on_level >= 3 even if openpilot / acc is not engaged. With the small change cancel commands will only be sent if acc / openpilot is engaged.

lukasloetkolben avatar Apr 12 '24 05:04 lukasloetkolben

Thanks for contributing to openpilot! In order for us to review your PR as quickly as possible, check the following:

  • Convert your PR to a draft unless it's ready to review
  • Read the contributing docs
  • Before marking as "ready for review", ensure:
    • the goal is clearly stated in the description
    • all the tests are passing
    • the change is something we merge
    • include a route or your device' dongle ID if relevant

github-actions[bot] avatar Apr 12 '24 05:04 github-actions[bot]