-
Notifications
You must be signed in to change notification settings - Fork 9
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
Port factorytest #88
Port factorytest #88
Conversation
Enabled for : Test mode, main menu, quick settings, gamepad Formatting
Move test assets to folder
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Function in header needs to be resolved. I'll try to do that, shouldn't be too bad.
* @brief Read, save, and validate accelerometer data | ||
* | ||
*/ | ||
void testReadAndValidateAccelerometer(void) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@cnlohr after this gets merged to main and you want to add your motion and calibration stuff, this is likely the right place to do it
Description
Ported factory test mode, modified it for new buzzers and touchpad.
Test Instructions
Tested on emulator and firmware
If any of these behaviors are incorrect, I can change them.
Ticket Links
Readiness Checklist
make format
to format the changesmake cppcheck
and checked thatcppcheck_result.txt
has no warnings for the changes/*! \file
comments with Design Philosophy, Usage, and Example sections for new headers.make docs
and checked thatdoxy_warnings.txt
has no warnings for the new code