-
Notifications
You must be signed in to change notification settings - Fork 285
Document common causes of loop overruns #3132
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
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||||
---|---|---|---|---|---|---|---|---|
|
@@ -46,6 +46,42 @@ Another way to debug your program is to use print statements in your code and vi | |||||||
|
||||||||
:doc:`NetworkTables </docs/software/networktables/networktables-intro>` can be used to share robot information with your debugging computer. :term:`NetworkTables` can be viewed with your favorite Dashboard or :ref:`OutlineViewer <docs/software/wpilib-tools/outlineviewer/index:OutlineViewer>`. One advantage of NetworkTables is that tools like :doc:`Shuffleboard </docs/software/dashboards/shuffleboard/getting-started/shuffleboard-tour>` can be used to graphically analyze the data. These same tools can then be used with same data to later provide an operator interface for your drivers. | ||||||||
|
||||||||
## Common Causes of Loop Overruns | ||||||||
|
||||||||
Loop overruns occur when the robot's periodic methods (``robotPeriodic()``, ``teleopPeriodic()``, etc.) take longer than 20ms to complete. When this happens, the Driver Station will display a warning like ``Loop time of 0.03s overrun`` and the robot code may behave unpredictably. Here are common causes: | ||||||||
|
||||||||
### Blocking Operations | ||||||||
|
||||||||
- **Thread.sleep() or wait()**: Never use blocking sleep or wait calls in periodic methods | ||||||||
- **Synchronous I/O**: Reading files, network operations, or other blocking I/O operations | ||||||||
- **Busy-wait loops**: Loops that repeatedly check a condition without yielding (e.g., ``while(!sensor.isReady()) {}``) | ||||||||
|
||||||||
### Excessive Computation | ||||||||
|
||||||||
- **Complex calculations in periodic methods**: Move expensive calculations to separate threads or spread them across multiple loop iterations | ||||||||
- **Large data structure operations**: Sorting, searching, or iterating over large arrays or lists | ||||||||
- **Unoptimized algorithms**: O(n²) or worse algorithms running on large datasets | ||||||||
|
||||||||
### Excessive Logging or Print Statements | ||||||||
|
||||||||
- **System.out.println() in loops**: Console output is slow, especially when called frequently | ||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||
- **Getting data to publish to NetworkTables**: While NetworkTables updates themselves are fast, retrieving complex data (e.g., vision processing results, large arrays) to publish can be slow | ||||||||
- **Excessive Shuffleboard updates**: Sending large amounts of data to the dashboard | ||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||
|
||||||||
### Hardware/Sensor Issues | ||||||||
|
||||||||
- **Synchronous CAN calls**: Some motor controller methods may block waiting for a response | ||||||||
- **I2C or SPI timeouts**: Faulty sensors or loose connections can cause communication timeouts | ||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||
- **USB device enumeration**: Plugging/unplugging USB devices during operation | ||||||||
|
||||||||
### Tips to Avoid Loop Overruns | ||||||||
|
||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||
- Use :doc:`Notifier </docs/software/convenience-features/scheduling-functions>` for operations that need precise timing independent of the main loop | ||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Perhaps |
||||||||
- Profile your code to identify slow sections (see :ref:`docs/software/advanced-gradlerio/profiling-with-visualvm:profiling with visualvm`) | ||||||||
- Remove or reduce print statements, especially in frequently-called code | ||||||||
- Cache values that are expensive to compute rather than recalculating every loop | ||||||||
- **Check the Driver Station log** to identify which periodic method is causing overruns. The log will show timestamps and which robot mode was active when the overrun occurred. Look for patterns - if overruns only happen during teleop, check ``teleopPeriodic()`` and subsystems used during teleop. If they occur consistently, check ``robotPeriodic()`` for code that runs regardless of mode. | ||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||
|
||||||||
## Learn More | ||||||||
|
||||||||
- To learn more about debugging with VS Code see this [link](https://code.visualstudio.com/docs/editor/debugging). | ||||||||
|
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.