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.

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

Test LogReader scenarios for auro_source function #32971

Closed
wants to merge 10,000 commits into from

Conversation

markopetkovic96
Copy link

@markopetkovic96 markopetkovic96 commented Jul 12, 2024

for #31223
After @adeebshihadeh's suggestion, tests are updated.

BBBmau and others added 30 commits May 28, 2024 14:45
* add devcontainer-rebuild workflow

* add shell

* add shell on ifs

* use scripts/retry.sh
Update fingerprints.py

KIA Sportage 5th GEN EUR ver
* no step but still send

* no this

* update diff
* revert me

* Revert "revert me"

This reverts commit 17d815d.

* duh we have timed!

* clean up

* use clocks

* re-initialize map on SSL handshake failure (time)

* this is fine, takes some time to init

* fix

* log errors like map renderer

* more clean up

full message is "loading style failed: SSL handshake failed"

* MOAR

* we still can't swap the token live

* mbgl has its own retries that never work, don't reinit multiple times at once

* simpler

* more

* whoops

* this works

* fix from merge

* rm

* fix cmt

* only an issue calling the function inside itself
* Added Arteon "3H" chassis code for Shooting Brake to VWCarDocs.

* Added new engine and transmission FW to Volkswagen fingerprints.py for Arteon Mk1.

* Apply suggestions from code review

* updates docs

---------

Co-authored-by: Shane Smiskol <[email protected]>
* cache get_torque_params

* switch to cache

* Update selfdrive/car/interfaces.py

---------

Co-authored-by: Shane Smiskol <[email protected]>
* pytest: discover everything in openpilot/

* ignore process replay

* remove --ignore=openpilot/

* ignore directories in root

* remove skip in selfdrive/test/process_replay

* use openpilot directory

* set openpilot directory for test_translations

* omit generated config.py files

* run ci

* cd into openpilot for test_translation

* &&

* remove rm main_test_en command

* remove echo

* increase shm-size to 2G

* remove rm

* test only cpp_files that are test_*.cc~

* add process_replay test skip

* use addopts in pyproject to ignore test_processes

* try --rootdir for pytest

* cleanups / add rootdir in PYTEST env

* add rm create_test_translations

* revert poetry.lock

* simplify

* no regen!

---------

Co-authored-by: Adeeb Shihadeh <[email protected]>
bump submodules

Co-authored-by: Vehicle Researcher <[email protected]>
* test_*.py files are no longer executable

* cleanup empty lines

* update precommit

* fixme

* enable

* build
* test

* little more

---------

Co-authored-by: Comma Device <[email protected]>
* this is not a test

* these are not tests!
* reduce friction coeff for ev6 and ioniq6

* Update ref

---------

Co-authored-by: Bruce Wayne <[email protected]>
sshane and others added 6 commits July 9, 2024 22:26
* formatting

* function signatures didn't match

* function signatures didn't match

* filtered and raw mean something totally different when it comes to params

filtered and raw mean something totally different when it comes to params

* cmt

* probably better for organization

* add todo

* STASH

* revert some stuff

* clean up

* oof
…aai#32949)

* improved PlotJuggler layout for lat accel controller

* help estimate steerActuatorDelay

* update Actuator Performance, add Vehicle Dynamics

* disable Y limits on actuator performance because sunny

* apply delay estimation feedback from harald

* gc extra custom series that PJ copied in

* label wordsmithing
* lat active/steer override: check up to now

* lint

* Update ref_commit
* check on either edge

* clean up

* clean up

* good thing I tested this first :P
Copy link
Contributor

github-actions bot commented Jul 12, 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

Copy link
Contributor

This PR has had no activity for 14 days. It will be automatically closed in 3 days if there is no activity.

@github-actions github-actions bot added the stale label Jul 30, 2024
@markopetkovic96
Copy link
Author

@adeebshihadeh

@github-actions github-actions bot removed the stale label Jul 31, 2024
Copy link
Contributor

github-actions bot commented Aug 9, 2024

This PR has had no activity for 9 days. It will be automatically closed in 2 days if there is no activity.

@github-actions github-actions bot added the stale label Aug 9, 2024
@markopetkovic96
Copy link
Author

@adeebshihadeh

@github-actions github-actions bot removed the stale label Aug 10, 2024
Copy link
Contributor

This PR has had no activity for 9 days. It will be automatically closed in 2 days if there is no activity.

@github-actions github-actions bot added the stale label Aug 19, 2024
Copy link
Contributor

This PR has been automatically closed due to inactivity. Feel free to re-open once activity resumes.

@github-actions github-actions bot closed this Aug 22, 2024
@adeebshihadeh adeebshihadeh reopened this Aug 22, 2024
@github-actions github-actions bot removed the stale label Aug 23, 2024
Copy link
Contributor

github-actions bot commented Sep 1, 2024

This PR has had no activity for 9 days. It will be automatically closed in 2 days if there is no activity.

@github-actions github-actions bot added the stale label Sep 1, 2024
@markopetkovic96
Copy link
Author

.

@maxime-desroches
Copy link
Contributor

This PR was closed because of a git history rewrite.
Please read #33399 for what to do with your fork and your PRs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.