r-transit / tidytransit

R package for working with GTFS data
https://r-transit.github.io/tidytransit/
150 stars 22 forks source link

optimize filter_stop_times #183

Closed polettif closed 2 years ago

polettif commented 2 years ago

Small optimiziation for filter_stop_times, reduces run time for a large feed (stop_times with 11 million rows) by ~40%.

codecov-commenter commented 2 years ago

Codecov Report

Merging #183 (679a3e9) into master (3559d9f) will decrease coverage by 0.00%. The diff coverage is 100.00%.

:exclamation: Current head 679a3e9 differs from pull request most recent head 8d1da7c. Consider uploading reports for the commit 8d1da7c to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #183      +/-   ##
==========================================
- Coverage   88.97%   88.96%   -0.01%     
==========================================
  Files          14       14              
  Lines        1170     1169       -1     
==========================================
- Hits         1041     1040       -1     
  Misses        129      129              
Impacted Files Coverage Δ
R/raptor.R 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 3559d9f...8d1da7c. Read the comment docs.