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

Greatly improve processing time #12

Merged
merged 13 commits into from
Aug 29, 2019
Merged

Greatly improve processing time #12

merged 13 commits into from
Aug 29, 2019

Conversation

hugomflavio
Copy link
Owner

Main changes:

  1. Detections are loaded with data.table::fread
  2. Detection timestamps are converted using fasttime::fastPOSIXct
  3. Substituted the for-loop for movement event creation with a more time-efficient method
  4. Movement data frames are now movement data tables
  5. Movement data tables now contain a Valid column, which informs the next steps of which movements should be considered for the analysis.
  6. The simplifyMovements function is now obsolete, and simple.movements are created based on the Valid column of the movements, not requiring integration of the results within status.df anymore.
  7. Check for detection before release has been moved to the beginning of the analysis, so the faulty detections can be removed before the creation of movement events.

@hugomflavio hugomflavio merged commit d5dba49 into master Aug 29, 2019
@hugomflavio hugomflavio deleted the devel branch August 29, 2019 10:02
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