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

[$500 bounty] Drive a loop in MetaDrive #30694

Closed
adeebshihadeh opened this issue Dec 12, 2023 · 2 comments · Fixed by #32308
Closed

[$500 bounty] Drive a loop in MetaDrive #30694

adeebshihadeh opened this issue Dec 12, 2023 · 2 comments · Fixed by #32308
Labels
bounty CI / testing good first issue Feasible for new contributers

Comments

@adeebshihadeh
Copy link
Contributor

adeebshihadeh commented Dec 12, 2023

Extend the MetaDrive test to drive a defined course and ensure it stays in the line. Requirements:

  • at least 1 minute of driving
  • passes reliably: 20+ times in a row
  • test fails if a reasonable user would have disengaged during the loop (drives out of lane, etc.)

Ideally, we can also configure the distance and/or runtime of the test with a simple parameter so that we can run a quick one in CI, and perhaps a longer version on master.

https://github.com/commaai/openpilot/blob/master/tools/sim/tests/test_sim_bridge.py

@adeebshihadeh adeebshihadeh pinned this issue Dec 12, 2023
@jnewb1
Copy link
Contributor

jnewb1 commented Dec 15, 2023

would also be cool if the route is logged as a github actions artifact that can be viewed in the openpilot tools if it fails

@adeebshihadeh
Copy link
Contributor Author

Good idea, added to the list

@jnewb1 jnewb1 moved this to Open in openpilot bounties Dec 18, 2023
@jnewb1 jnewb1 linked a pull request Dec 20, 2023 that will close this issue
@jnewb1 jnewb1 removed a link to a pull request Dec 20, 2023
@adeebshihadeh adeebshihadeh moved this from Open to Locked in openpilot bounties Mar 3, 2024
@adeebshihadeh adeebshihadeh moved this from Locked to Open in openpilot bounties Apr 24, 2024
@adeebshihadeh adeebshihadeh moved this from Open to Locked in openpilot bounties May 19, 2024
@github-project-automation github-project-automation bot moved this from Locked to Done in openpilot bounties May 22, 2024
@adeebshihadeh adeebshihadeh unpinned this issue May 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bounty CI / testing good first issue Feasible for new contributers
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants