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.

Sign up for GitHub

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

Jump to bottom

show PLA edits #61

Open
wants to merge 50 commits into
base: FrogPilot-VW-NEW
Choose a base branch
from
Open

show PLA edits #61

wants to merge 50 commits into from

Conversation

dkiiv
Copy link
Owner

@dkiiv dkiiv commented Sep 4, 2024

for testing

dkiiv added 30 commits July 18, 2024 13:38
@dkiiv
init PLA_1 commit
68364de
@dkiiv
make CRC compliant
b1c5b5a
@dkiiv
fix typo
49cbfeb
@dkiiv
fix angle rate limit?
1551e7d
@dkiiv
Update carcontroller.py
dbf3d50
@dkiiv
Update interface.py
46cfd6e
@dkiiv
Update interface.py
99aa1a4
@dkiiv
Update interface.py
afedb06
@dkiiv
hail mary
522f777
@dkiiv
fix dbc
a2b97ad
@dkiiv
try hacking canValid out
cbc822b
@dkiiv
undo failed canValid hack
0a72053
@dkiiv
configure SteerControlType
6e7ed72
@dkiiv
add debug print
34dc1d8
@dkiiv
trying printing var to file
17b3b00
@dkiiv
print whole array before being iterated
77bb70e
@dkiiv
update the print
97f33b3
@dkiiv
remove debug print
479537c
@dkiiv
re-add new_actuators, thanks sunny!
471f2d7
@dkiiv
restore formatting
6b226c9
@dkiiv
fix PLA msg size
2d4e187
@dkiiv
mimick EPS steeringAngle when not enabled
fe9cd16
@dkiiv
remove long pulse logic
2289b00
@dkiiv
mimick OEM PLA engagement behavior, handle driverOverride
42e4f63
@dkiiv
ensure angle req is equal to normal steering angle during entry sequence
9ea2392
@dkiiv
fix PLA_entryCounter, fix panda reject?
635b36e
@dkiiv
delay PLA angle by 1 more frame if not engaged
d19445e
@dkiiv
lord forgive me. i swear it's just for testing.
a59c8a1
@dkiiv
update checksum routine + algo
db22702 
* artificially mimick OEM entry behavior
@dkiiv
update PLA statusLast location
696789f
dkiiv added 20 commits July 21, 2024 16:10
@dkiiv
move counter to carport, try to steer at standstill
1f0001b
@dkiiv
remove artificial 00, add rest of PLA signal
3b26875
@dkiiv
fix crc on PLA in panda
01681e1
@dkiiv
;
71b0d32
@dkiiv
undo my crc edit in panda
e036ac3
@dkiiv
the unholy edit..
30bf3fc
@dkiiv
setup steerFaultTemp
40a1d39
@dkiiv
moving checksum+counter back to panda/dbc
764d7b6
@dkiiv
remove panda hack and orphaned variable
8df6239
@dkiiv
update PLA_1 message length
13adb1b
@dkiiv
forward car's steeringSign into mPLA_1
db9ca47
@dkiiv
update PLA length, fill signal, add ESP_Status, fix checksum
fa5ce97
@dkiiv
hack OP up some more, controls in reverse?
f3fb156
@dkiiv
increase rate limit for standstill only
4892451
@dkiiv
init gk_1 spam
5719af7
@dkiiv
fix panda canID, update GK_1 spammer
2c44c65
@dkiiv
put GK step where it belongs.. hehe oops
dc4a126
@dkiiv
delay PLA_1 by 1 frame
3792ca2
@dkiiv
revert me if the intended behavior breaks
7a123ae
@dkiiv
automate bitbang, handle failed entry
b8e99c5
@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 4, 2024

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dkiiv @FLcruising