diff --git a/CHANGELOG.md b/CHANGELOG.md index d4ba8226934..fcbbbbb36ec 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -73,6 +73,7 @@ - FIXED: Ensure required file check in osrm-routed is correctly enforced. [#6655](https://github.com/Project-OSRM/osrm-backend/pull/6655) - FIXED: Correct HTTP docs to reflect summary output dependency on steps parameter. [#6655](https://github.com/Project-OSRM/osrm-backend/pull/6655) - ADDED: Extract prerelease/build information from package semver [#6839](https://github.com/Project-OSRM/osrm-backend/pull/6839) + - ADDED: Handle conditional access tags for date ranges [#6984](https://github.com/Project-OSRM/osrm-backend/pull/6984) - Profiles: - FIXED: Bicycle and foot profiles now don't route on proposed ways [#6615](https://github.com/Project-OSRM/osrm-backend/pull/6615) - Routing: diff --git a/features/car/access.feature b/features/car/access.feature index 0bebd2589e0..4d7f278f469 100644 --- a/features/car/access.feature +++ b/features/car/access.feature @@ -305,3 +305,16 @@ Feature: Car - Restricted access | primary | psv | | | primary | no | | | primary | customers | x | + + Scenario: Car - Conditional restrictions + Then routability should be + | highway | motor_vehicle | motor_vehicle:conditional | bothw | + | primary | yes | | x | + | primary | yes | no @ 2002 Jan 7 - 2002 Feb 8 | x | + | primary | yes | no @ 2002 Jan 07 - 2002 Feb 08 | x | + | primary | yes | no @ 2090 Jan 7 - 2100 Feb 8 | x | + | primary | yes | no @ 2020 Jan 7 - 2050 Feb 8 | | + | primary | yes | no @ 2020 Jan 07 - 2050 Feb 08 | | + | primary | yes | no @ (2020 Jan 7 - 2050 Feb 8) | | + | primary | yes | no @ foo - bar | x | + | primary | yes | foo | x | diff --git a/profiles/lib/conditional_access.lua b/profiles/lib/conditional_access.lua new file mode 100644 index 00000000000..6a857c54f77 --- /dev/null +++ b/profiles/lib/conditional_access.lua @@ -0,0 +1,80 @@ +-- Handle conditional access tags as described in the OSM wiki: +-- https://wiki.openstreetmap.org/wiki/Conditional_restrictions + +-- Note that we only handle conditional tags for a date range, +-- meant to be used for temporary restrictions, typically due to +-- construction. We also require the date range to be at least a +-- week long + + + +ConditionalAccess = {} + + +local function parse_conditional_access(way, key) + local conditional = way:get_value_by_key(key .. ':conditional') + if not conditional then + return nil + end + + -- Examples of conditional tags: + -- "no @ (2018 May 22-2018 Oct 7)" or + -- "no @ 2018 Jun 01 - 2018 Jul 23" + local condition, time_range = conditional:match("([^@]+)@(.+)") + if not condition or not time_range then + return nil + end + + local start_date_str, end_date_str = time_range:match("([^-]+)-(.+)") + if not start_date_str or not end_date_str then + return nil + end + + local function parse_date(date_str) + local year, month, day = date_str:match("(%d+)%s+(%a+)%s+(%d+)") + + local month_names = { + Jan = 1, Feb = 2, Mar = 3, Apr = 4, May = 5, Jun = 6, + Jul = 7, Aug = 8, Sep = 9, Oct = 10, Nov = 11, Dec = 12 + } + month = month_names[month] + if not year or not month or not day then + return nil + end + + local numericYear = tonumber(year) + local numericDay = tonumber(day) + if numericYear and numericDay then + return os.time({ year = numericYear, month = month, day = numericDay }) + else + return nil + end + end + + local start_date = parse_date(start_date_str) + local end_date = parse_date(end_date_str) + local current_date = os.time() + + -- Require start and end date to be more than a week apart + if not start_date or not end_date or end_date - start_date < 60 * 60 * 24 * 7 then + return nil + end + + if current_date >= start_date and current_date <= end_date then + return condition:match("%S+") + else + return nil + end + end + +function ConditionalAccess.parse_by_set(way, keys) + for i, key in ipairs(keys) do + local conditional = parse_conditional_access(way, key) + if conditional then + return conditional + end + end + return nil +end + +return ConditionalAccess diff --git a/profiles/lib/way_handlers.lua b/profiles/lib/way_handlers.lua index b1341023509..5b982d41106 100644 --- a/profiles/lib/way_handlers.lua +++ b/profiles/lib/way_handlers.lua @@ -9,6 +9,7 @@ local set_classification = require("lib/guidance").set_classification local get_destination = require("lib/destination").get_destination local Tags = require('lib/tags') local Measure = require("lib/measure") +local ConditionalAccess = require("lib/conditional_access") WayHandlers = {} @@ -243,6 +244,13 @@ function WayHandlers.access(profile,way,result,data) data.forward_access, data.backward_access = Tags.get_forward_backward_by_set(way,data,profile.access_tags_hierarchy) + -- check for conditional access (roads that are temporarily closed, etc.) + local conditional = ConditionalAccess.parse_by_set(way,profile.access_tags_hierarchy) + if conditional then + data.forward_access = conditional + data.backward_access = conditional + end + -- only allow a subset of roads to be treated as restricted if profile.restricted_highway_whitelist[data.highway] then if profile.restricted_access_tag_list[data.forward_access] then