Embed Notice

HTML Code

<blockquote style="position: relative; padding-left: 55px;"><section><a href="https://chaos.social/users/jonty/statuses/113827628811181120">Jonty Wareing (jonty@chaos.social)'s status on Wednesday, 15-Jan-2025 02:03:59 JST</a><a href="https://chaos.social/@jonty" title="jonty@chaos.social"><img src="https://gnusocial.jp/avatar/29018-48-20221117074442.webp" width="48" height="48" alt="Jonty Wareing" style="position: absolute; left: 0; top: 0;">Jonty Wareing</a></section><article><p>Things I regret: Looking at github issues for the open-source self-driving car conversion kit.</p><p>Thousands of people use these things on public roads.</p><p><a href="https://github.com/commaai/openpilot/issues/34346" rel="nofollow noreferrer">https://github.com/commaai/openpilot/issues/34346</a></p></article><footer><a rel="bookmark" href="https://gnusocial.jp/conversation/4373804#notice-8553821">In conversation</a><time datetime="2025-01-15T02:03:59+09:00" title="Wednesday, 15-Jan-2025 02:03:59 JST">about 4 days ago</time> <span>from <span><a href="https://chaos.social/@jonty/113827628811181120" rel="external" title="Sent from chaos.social via ActivityPub">chaos.social</a></span></span><a href="https://chaos.social/@jonty/113827628811181120">permalink</a><h4>Attachments</h4><ol><li><label><a rel="external" href="https://gnusocial.jp/attachment/3914371">Screenshot of a github issue titled "22 Acura RDX - steering disables under 45mph".It reads:Steering is completely disabled below 45mph on release branch. There is no visual warning saying it is disabled, the car just drives in a straight line until it about to go offroad and reports a manual steering required warning.I have traced the RCA to the release branch requiring factory ACC for longitudinal control. The factory ACC disables LKAS below 45mph, and therefore it appears the car ignores all comma steering commands. If install the latest devel branch, and enable comma longitudinal control instead of factory, then steering works at all speeds.I would expect some type of warning under 45mph, rather than a green path that the car just ignores. Or an preferred fix is to default release branch to comma longitudinal for this car model, either always or whenever under 45mph.</a></label><br><a href="https://assets.chaos.social/media_attachments/files/113/827/592/092/106/911/original/dfba7fb14df05d0c.png" rel="external">https://assets.chaos.social/media_attachments/files/113/827/592/092/106/911/original/dfba7fb14df05d0c.png</a></li></ol></footer></blockquote>

Corresponding Notice

Embed this notice
Jonty Wareing (jonty@chaos.social)'s status on Wednesday, 15-Jan-2025 02:03:59 JST Jonty Wareing
Things I regret: Looking at github issues for the open-source self-driving car conversion kit.
Thousands of people use these things on public roads.
https://github.com/commaai/openpilot/issues/34346
In conversationabout 4 days ago from chaos.socialpermalink
Attachments
1. Screenshot of a github issue titled "22 Acura RDX - steering disables under 45mph". It reads: Steering is completely disabled below 45mph on release branch. There is no visual warning saying it is disabled, the car just drives in a straight line until it about to go offroad and reports a manual steering required warning. I have traced the RCA to the release branch requiring factory ACC for longitudinal control. The factory ACC disables LKAS below 45mph, and therefore it appears the car ignores all comma steering commands. If install the latest devel branch, and enable comma longitudinal control instead of factory, then steering works at all speeds. I would expect some type of warning under 45mph, rather than a green path that the car just ignores. Or an preferred fix is to default release branch to comma longitudinal for this car model, either always or whenever under 45mph.
  https://assets.chaos.social/media_attachments/files/113/827/592/092/106/911/original/dfba7fb14df05d0c.png

Public

Embed Notice

HTML Code

Corresponding Notice