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

RT #31930

Closed
wants to merge 18 commits into from
Closed

RT #31930

wants to merge 18 commits into from

Conversation

fredyshox
Copy link
Contributor

No description provided.

@sshane
Copy link
Contributor

sshane commented Mar 20, 2024

the model is eating openpilot!

@foodbandlt
Copy link
Contributor

I know this is a draft/experimental PR, but I had some spare time and tried it with extreme supervision 😅 Here's some routes from commit 576ebf1 and onward, some have bookmarks, all on chill mode:

  • b4f83330b2d59205/00000048--06873ca286
  • b4f83330b2d59205/00000049--b841213b4a
  • b4f83330b2d59205/0000004a--5d86e92a97
  • b4f83330b2d59205/0000004c--3b053b2522
  • b4f83330b2d59205/0000004e--72a70916aa

On the most recent commit, it seems to react poorly to both up and down hill and things over the road, even at night! Since the UI commits, when this happens there's no lead chevron on screen. Lots of gas overriding on the highway to not drop 20mph due to an overpass or overhanging sign! Very cool work, excited to see where it goes 😄

@nworb-cire
Copy link
Contributor

I find this to be driving quite well on my Volt without radar. The VOACC is better than e2e for the first time in a while.

Here are a few routes if you want them:

  • 806907cd33ef2647/00000107--7de80f3ce9
  • 806907cd33ef2647/00000108--c7b2565fd2
  • 806907cd33ef2647/00000109--02cd3821e6
  • 806907cd33ef2647/0000010f--e7646a2cb6
  • 806907cd33ef2647/00000111--171052fde4

In that first route, you can see a bit of odd behaviour wherein chill mode slows for some stop signs. Chill mode also occasionally stops uncomfortably close to lead cars IMO.

@sshane sshane deleted the rt-model branch May 13, 2024 04:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants