Skip to content

DO NOT MERGE - Verify APPS/BSE Violation Occurences - DO NOT MERGE#512

Closed
dchansen06 wants to merge 4 commits into
mainfrom
IllegalAppsBseViolationSound
Closed

DO NOT MERGE - Verify APPS/BSE Violation Occurences - DO NOT MERGE#512
dchansen06 wants to merge 4 commits into
mainfrom
IllegalAppsBseViolationSound

Conversation

@dchansen06

@dchansen06 dchansen06 commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

Bug Test APPS/BSE

Problem and Scope

Suspect transient APPS/BSE violations are causing issues

Description

If the apps/bse violation occurs then sound the buzzer until it finishes plus 100 ms

Gotchas and Limitations

Caution

This PR stands in direct violation to

EV.9.7.3 The vehicle must not make other sounds similar to the Ready to Drive Sound.
As a result it should NOT be merged without changing

Testing

  • HOOTL testing
  • HITL testing
  • Human tested

Testing Details

Driver will tell us if they hear a sound

Larger Impact

Try and identify momentary APPS/BSE violation

Additional Context and Ticket

See Discord

…hes plus 100 ms

Signed-off-by: Daniel Hansen <dchansen06@gmail.com>
@dchansen06 dchansen06 self-assigned this Jun 7, 2026
@dchansen06 dchansen06 added Enhancement New feature or request 2 PRIORITY Important and a priority, but less than URGENT Small Fry Something that is small, could include bug fixes or smaller changes Sensors Something involved with or related to sensors Peripheral Related to or involving a peripheral including abstractions Boards Related to or involving any physical boards Bug Something is doing a thing but doing it wrong or otherwise incorrectly labels Jun 7, 2026
@dchansen06 dchansen06 added this to the GR26 milestone Jun 7, 2026
@dchansen06 dchansen06 changed the title DO NOT MERGE - Verify APPS/BSE Violation Instances DO NOT MERGE - Verify APPS/BSE Violation Occurences Jun 7, 2026
@dchansen06 dchansen06 changed the title DO NOT MERGE - Verify APPS/BSE Violation Occurences DO NOT MERGE - Verify APPS/BSE Violation Occurences - DO NOT MERGE Jun 7, 2026
@dchansen06

dchansen06 commented Jun 9, 2026

Copy link
Copy Markdown
Contributor Author

Closed in favor of #504 which now has the good changes

@dchansen06 dchansen06 closed this Jun 9, 2026
@dchansen06 dchansen06 deleted the IllegalAppsBseViolationSound branch June 9, 2026 04:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2 PRIORITY Important and a priority, but less than URGENT Boards Related to or involving any physical boards Bug Something is doing a thing but doing it wrong or otherwise incorrectly Enhancement New feature or request Peripheral Related to or involving a peripheral including abstractions Sensors Something involved with or related to sensors Small Fry Something that is small, could include bug fixes or smaller changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants