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

Update tracking_metrics.py #13

Merged
merged 1 commit into from
Jun 5, 2024
Merged

Update tracking_metrics.py #13

merged 1 commit into from
Jun 5, 2024

Conversation

lanery
Copy link
Collaborator

@lanery lanery commented Jun 4, 2024

Change definition of max_sprint_length to be calculated as a rolling sum (to better match the tracks in the animations). Facilitates interpretation of this measurement. num_direction_changes (and therefore also pivot_rate) are also changed to be defined as rolling averages, but more so to suppress noise in the trajectory.

Pretty much summed up in the commit message

Change definition of `max_sprint_length` to be calculated as a rolling sum (to better
match the tracks in the animations). Facilitates interpretation of this measurement.
`num_direction_changes` (and therefore also `pivot_rate` are also changed to be defined
as rolling averages, but more so to suppress noise in the trajectory.

Have not yet taken the time to write integration tests, but have re-analyzed all of the useful motility data we have with these new definitions.

PR checklist

  • Describe the changes you've made.
  • Describe any tests you have conducted to confirm that your changes behave as expected.
  • If you've added new functionality, make sure that the documentation is updated accordingly.

Change definition of `max_sprint_length` to be calculated as a rolling sum (to better match the tracks in the animations). Facilitates interpretation of this measurement. `num_direction_changes` (and therefore also `pivot_rate` are also changed to be defined as rolling averages, but more so to suppress noise in the trajectory.
@lanery lanery requested a review from keithchev June 4, 2024 19:03
@lanery lanery merged commit aed3d86 into main Jun 5, 2024
1 check passed
@lanery lanery deleted the rl/adjust-motility-metrics branch June 5, 2024 00:14
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.

2 participants