search

kaijagahm / vultureUtils

Utility functions for working with vulture data
Other
4 stars 0 forks source link

filterLocs: make lower bound inclusive #119

Open kaijagahm opened 1 year ago

kaijagahm commented 1 year ago

speedThreshLower should be an inclusive boundary: >=, not >. Do not change the speedThreshUpper filter to be inclusive, because then that would include individuals going exactly, say, 5 m/s in both flight and feeding edges, which would not be appropriate.

kaijagahm commented 1 year ago

@ryannguyen6392 the actual change here is extremely trivial, just replacing > with >= in filterLocs. But I want to make sure we're doing this correctly with the testing. Can you walk me through how we should implement this change, which should, and will, change the output for the testing dataset?