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

Improve comma three offroad thermals #23144

Closed
2 of 3 tasks
pd0wm opened this issue Dec 6, 2021 · 6 comments
Closed
2 of 3 tasks

Improve comma three offroad thermals #23144

pd0wm opened this issue Dec 6, 2021 · 6 comments

Comments

@pd0wm
Copy link
Contributor

pd0wm commented Dec 6, 2021

  • Switch to powersave governor when offroad
  • Look into Quectel power savings
  • Lower fan control setpoint
@pd0wm pd0wm added this to the 0.8.12 milestone Dec 6, 2021
@pd0wm pd0wm changed the title Improve comma three thermals Improve comma three offroad thermals Dec 6, 2021
@pd0wm pd0wm removed this from the 0.8.12 milestone Dec 14, 2021
@adeebshihadeh
Copy link
Contributor

I think we should also re-evaulate the threshold to go onroad. Currently it's set at 79.5C, but a device that goes onroad at that temperature will either narrowly avoid going into the red band or immediately jump into it. We'll either need to keep the device cooler while it's offroad or build a better experience when we don't immediately go onroad.

image

@adeebshihadeh adeebshihadeh added this to the 0.8.15 milestone Apr 28, 2022
@jyoung8607
Copy link
Collaborator

3cfdec54aa035f3f|2022-05-29--16-38-08 is an example of the problem Adeeb described, a startup from just below the offroad danger temp that goes just into the red a short time after, resulting in a very long delay before openpilot was engageable.

@Hubblesphere
Copy link

Lapse in overheat protection. Off road overheated able to engage temporarily when going onroad before detecting an overheated condition again.

Route: 8d8427a2116eace2|2022-07-22--13-00-54--0

@jyoung8607
Copy link
Collaborator

Another example: 3cfdec54aa035f3f|2022-08-24--15-21-31

@adeebshihadeh adeebshihadeh removed this from the 0.8.17 milestone Nov 11, 2022
@goodchoom
Copy link
Contributor

When the temperature outside is 70°F or hotter and my car has been sitting outside for an extended period of time, I always remote start my car with the A/C set to LO several minutes in advance so by the time I enter my car it's comfortable for me to drive. Despite the cabin having cooled down, I am sometimes unable to engage openpilot because it went onroad just below the offroad danger temp and then overheated while the cabin was hot. When this happens I have to wait/drive manually for several more minutes until the device cools down enough to be able to engage openpilot again.

Here is a route of this issue occuring: 4f3e4f2f6d94e940|2023-04-20--12-27-08--0

@adeebshihadeh
Copy link
Contributor

When the temperature outside is 70°F or hotter and my car has been sitting outside for an extended period of time, I always remote start my car with the A/C set to LO several minutes in advance so by the time I enter my car it's comfortable for me to drive. Despite the cabin having cooled down, I am sometimes unable to engage openpilot because it went onroad just below the offroad danger temp and then overheated while the cabin was hot. When this happens I have to wait/drive manually for several more minutes until the device cools down enough to be able to engage openpilot again.

Here is a route of this issue occuring: 4f3e4f2f6d94e940|2023-04-20--12-27-08--0

Recent releases have addressed the onroad overheating. Let us know if you still have issues.

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

No branches or pull requests

6 participants