Skip to content

Sentry mode: automatically records and plays noise when bumped - #553

Merged
sshane merged 91 commits into
SA-masterfrom
SA-sentry-mode
Jan 21, 2022
Merged

Sentry mode: automatically records and plays noise when bumped#553
sshane merged 91 commits into
SA-masterfrom
SA-sentry-mode

Conversation

@sshane

@sshane sshane commented Jan 11, 2022

Copy link
Copy Markdown
Owner
  • Unarmed when locked/unlocked (just resets timer)
  • Unarmed when doors are unlocked
  • Fix interaction between thermald and sentryd (thermald needs to respect sentryd's onroad status and set params and power mode accordingly).
    • Since sentryd is 100hz and thermal much lower, we'd have to wait for thermald to set the powersave mode before going onroad. Instead, we set the powersave mode in sentryd and stop thermald from changing it when sentry mode is active.
    • In the future, we should let thermald handle everything from starting processes to setting the powersave mode.

@sshane sshane added the feature label Jan 11, 2022
@sshane
sshane force-pushed the SA-sentry-mode branch 2 times, most recently from 97ea906 to 660bd29 Compare January 12, 2022 00:12
@sshane sshane changed the title Sentry mode: automatically records and plays noise when bump is detected Sentry mode: automatically records and plays noise when bumped Jan 12, 2022
@sshane
sshane merged commit 4607465 into SA-master Jan 21, 2022
@sshane
sshane deleted the SA-sentry-mode branch January 21, 2022 03:26
Moodkiller added a commit to Moodkiller/sunnypilot that referenced this pull request Jan 29, 2022
Source: sshane/openpilot#553

Fix logging error

Fix for undeclared identifier (offroadTransition)

Return of the offroadTransition Part.1

Is this the only error (missing default argument)

Remove the bool instead

return bool

Try setting an argument on `started_sentry`

Fix function again
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant