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

Add Timing Instrumentation to has_trip_ended Function #997

Conversation

TeachMeTW
Copy link
Contributor

Summary

Adds timing instrumentation to the has_trip_ended function, improving observability and allowing for detailed performance monitoring of key operations.

Changes

  1. Timing Instrumentation:
    • Added timing measurements for the following operations within the has_trip_ended function:
      • Time Calculations: Measures the computation time for variables like timeDelta, distDelta, and speedDelta.
      • Motion Check: Tracks the time taken to evaluate ongoing motion in the specified time range.
      • Distance Threshold Check: Times the evaluation of last5MinsDistances and last10PointsDistances against their thresholds.

…ring

- Introduced timers to measure the duration of key operations
@TeachMeTW TeachMeTW force-pushed the Add_has_trip_ended_instrumentation branch from bfeb3f9 to 199fe4e Compare December 3, 2024 21:38
@TeachMeTW TeachMeTW closed this Dec 9, 2024
@TeachMeTW
Copy link
Contributor Author

Was part of the bottom 80%, no longer needed

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

Successfully merging this pull request may close these issues.

1 participant