closes #342

r-package/NEWS.md

@@ -1,5 +1,9 @@
 # r5r 1.0.1
 
+**Minor changes**
+
+- The default value of `time_window` is not set to 10 minutes in all functions to avoid weird results reported upstream in R5. Closes [#342](https://github.com/ipeaGIT/r5r/issues/342).
+
 **Bug fixes**
 
 * Updated to R5 version 6.9. This fixed a few bugs upstream, one of which often prevented users to build a network using cropped OSM data. Closes [#325](https://github.com/ipeaGIT/r5r/issues/325).

r-package/R/accessibility.R

@@ -139,7 +139,7 @@ accessibility <- function(r5r_core,
                           mode = "WALK",
                           mode_egress = "WALK",
                           departure_datetime = Sys.time(),
-                          time_window = 1L,
+                          time_window = 10L,
                           percentiles = 50L,
                           decay_function = "step",
                           cutoffs = NULL,

r-package/R/detailed_itineraries.R

@@ -15,8 +15,8 @@
 #' @template fare_structure
 #' @template max_fare
 #' @param time_window An integer. The time window in minutes for which `r5r`
-#'   will calculate multiple itineraries departing each minute. Defaults to 1
-#'   minute. If the same sequence of routes appear in different minutes of the
+#'   will calculate multiple itineraries departing each minute. Defaults to 10
+#'   minutes. If the same sequence of routes appear in different minutes of the
 #'   time window, only the fastest of them will be kept in the output. This
 #'   happens because the result is not aggregated by percentile, as opposed to
 #'   other routing functions in the package. Because of that, the output may
@@ -90,7 +90,7 @@ detailed_itineraries <- function(r5r_core,
                                  mode = "WALK",
                                  mode_egress = "WALK",
                                  departure_datetime = Sys.time(),
-                                 time_window = 1L,
+                                 time_window = 10L,
                                  suboptimal_minutes = 0L,
                                  fare_structure = NULL,
                                  max_fare = Inf,

r-package/R/expanded_travel_time_matrix.R

@@ -90,7 +90,7 @@ expanded_travel_time_matrix <- function(r5r_core,
                                         mode = "WALK",
                                         mode_egress = "WALK",
                                         departure_datetime = Sys.time(),
-                                        time_window = 1L,
+                                        time_window = 10L,
                                         breakdown = FALSE,
                                         max_walk_time = Inf,
                                         max_bike_time = Inf,

r-package/R/pareto_frontier.R

@@ -92,7 +92,7 @@ pareto_frontier <- function(r5r_core,
                             mode = c("WALK", "TRANSIT"),
                             mode_egress = "WALK",
                             departure_datetime = Sys.time(),
-                            time_window = 1L,
+                            time_window = 10L,
                             percentiles = 50L,
                             max_walk_time = Inf,
                             max_bike_time = Inf,

r-package/R/travel_time_matrix.R

@@ -112,7 +112,7 @@ travel_time_matrix <- function(r5r_core,
                                mode = "WALK",
                                mode_egress = "WALK",
                                departure_datetime = Sys.time(),
-                               time_window = 1L,
+                               time_window = 10L,
                                percentiles = 50L,
                                fare_structure = NULL,
                                max_fare = Inf,

r-package/man/roxygen/templates/time_window_related_args.R

@@ -1,6 +1,6 @@
 #' @param time_window An integer. The time window in minutes for which `r5r`
 #'   will calculate multiple travel time matrices departing each minute.
-#'   Defaults to 1 minute. By default, the function returns the result based on
-#'   median travel times, but the user can set the `percentiles` parameter to
+#'   Defaults to 10 minutes. By default, the function returns the result based
+#'   on median travel times, but the user can set the `percentiles` parameter to
 #'   extract more results. Please read the time window vignette for more details
 #'   on its usage `vignette("time_window", package = "r5r")`