Skip to content

feat: filter trails by activity + backfill trail activities #456

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.

Sign up for GitHub

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

Closed
fatherlinux wants to merge 1 commit into from
+65 −3
Closed

feat: filter trails by activity + backfill trail activities #456

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
34 changes: 34 additions & 0 deletions backend/migrations/075_backfill_trail_activities.sql
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
-- Migration 075: Backfill and correct trail activities so the activity filter
-- (e.g. selecting "Biking") only surfaces trails that actually allow that activity.
--
-- Three parts, all idempotent (guarded so re-runs are no-ops and so later admin
-- edits are not clobbered once a row no longer matches the known-bad state):
-- 1. Untagged trails -> 'Hiking' (most are CVNP/Metro foot trails).
-- 2. Drop 'Biking' from 11 footpaths that were mis-tagged as bikeable.
-- 3. Bikeable multi-use trails on the allowlist (Towpath, Bike & Hike, Gateway,
-- etc.) keep their existing 'Hiking, Biking' tags — no change needed.

BEGIN;

-- 1. Untagged trails -> 'Hiking'
UPDATE pois
SET primary_activities = 'Hiking'
WHERE 'trail' = ANY(poi_roles)
AND coalesce(deleted, false) = false
AND coalesce(nullif(trim(primary_activities), ''), '') = '';

-- 2. Remove 'Biking' from hiking-only footpaths that were incorrectly tagged.
-- Element-wise: split on commas, drop the 'Biking' item, rejoin. Guarded by id list
-- + "still contains Biking" (Postgres word boundaries are \m \M, NOT \b), so it
-- fires once and re-runs as UPDATE 0. Gateway Trail (West Creek paved connector,
-- id 1018) and the paved/limestone multi-use trails are intentionally excluded.
UPDATE pois
SET primary_activities = (
SELECT string_agg(trim(part), ', ')
FROM unnest(string_to_array(primary_activities, ',')) AS part
WHERE lower(trim(part)) <> 'biking'
)
WHERE id IN (974, 1021, 1036, 1045, 1050, 1057, 1068, 1074, 1089, 1099, 1095)
AND primary_activities ~* '\mBiking\M';

COMMIT;
6 changes: 3 additions & 3 deletions frontend/src/components/Map.jsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ import { createPortal } from 'react-dom';
import { MapContainer, TileLayer, Marker, Tooltip, useMap, GeoJSON, useMapEvents, CircleMarker } from 'react-leaflet';
import L from 'leaflet';
import VirtualPoiCreator from './VirtualPoiCreator';
import { getDestinationIconTypeFromConfig, poiMatchesActivityForTypes, matchesWholeWord } from '../utils/iconUtils';
import { getDestinationIconTypeFromConfig, poiMatchesActivityForTypes, trailPassesActivityFilter, matchesWholeWord } from '../utils/iconUtils';
import { useTrip } from '../hooks/useTrip';
import { useNavigate } from 'react-router-dom';
import { generateSlug } from './sidebar/helpers';
Expand Down Expand Up @@ -635,7 +635,7 @@ function MapBoundsTracker({ destinations, visibleTypes, getDestinationIconType,
// Title search matches across all linear types, ignoring layer toggles
isLayerVisible = feature.name?.toLowerCase().includes(search);
} else if (feature.poi_roles?.includes('trail')) {
isLayerVisible = showTrails;
isLayerVisible = showTrails && trailPassesActivityFilter(feature, visibleTypes, iconConfig);
} else if (feature.poi_roles?.includes('river')) {
isLayerVisible = showRivers;
} else if (feature.poi_roles?.includes('water_taxi')) {
Expand Down Expand Up @@ -1495,7 +1495,7 @@ function Map({ destinations, selectedPoi, selectedIsLinear, onSelectPoi, isAdmin
// regardless of its layer toggle, and hide non-matches.
const isVisible = searchQuery
? feature.name?.toLowerCase().includes(searchQuery.toLowerCase())
: ((feature.poi_roles?.includes('trail') && showTrails) ||
: ((feature.poi_roles?.includes('trail') && showTrails && trailPassesActivityFilter(feature, visibleTypes, iconConfig)) ||
(feature.poi_roles?.includes('river') && showRivers) ||
(feature.poi_roles?.includes('water_taxi') && showWaterTaxis) ||
(feature.poi_roles?.includes('boundary') && visibleBoundaries.has(feature.id)));
Expand Down
28 changes: 28 additions & 0 deletions frontend/src/utils/iconUtils.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,34 @@ export function poiMatchesActivityForTypes(poi, visibleTypes, iconConfig) {
return false;
}

/**
* True when the legend is narrowed to a subset of activity-bearing types
* (e.g. just "Biking") rather than showing everything. The Trails layer is only
* refined by activity while this is true; with all (or no) activity types
* selected it stays all-or-nothing, preserving plain "show me the trails" browsing.
*/
export function isActivityFilterActive(visibleTypes, iconConfig) {
if (!iconConfig || iconConfig.length === 0) return false;
let activityTypes = 0, selected = 0;
for (const icon of iconConfig) {
if (icon.enabled === false || !icon.activity_fallbacks) continue;
activityTypes++;
if (visibleTypes.has(icon.name)) selected++;
}
return selected > 0 && selected < activityTypes;
}

/**
* Whether a trail (linear feature) should render given the current activity
* narrowing. Untagged trails always show, so a missing tag never silently hides a
* trail. (Selecting "Biking" hides hiking-only trails.)
*/
export function trailPassesActivityFilter(feature, visibleTypes, iconConfig) {
if (!isActivityFilterActive(visibleTypes, iconConfig)) return true;
if (!(feature.primary_activities || '').trim()) return true;
return poiMatchesActivityForTypes(feature, visibleTypes, iconConfig);
}
Comment on lines +70 to +90
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot May 31, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

Performance Bottleneck & Defensive Programming

  1. Performance Bottleneck: isActivityFilterActive is called inside trailPassesActivityFilter for every single trail feature (e.g., 179 trails) during map rendering and bounds tracking. Since isActivityFilterActive only depends on visibleTypes and iconConfig (which are constant for a given render pass), this results in redundant loops over iconConfig hundreds of times per render. We can optimize this by caching the last computed result using reference equality checks on visibleTypes and iconConfig.
  2. Defensive Programming: Added defensive checks to ensure visibleTypes is a valid Set (or has a has method) and feature is non-null before accessing their properties.
let lastVisibleTypes = null;
let lastIconConfig = null;
let lastResult = false;

export function isActivityFilterActive(visibleTypes, iconConfig) {
  if (visibleTypes === lastVisibleTypes && iconConfig === lastIconConfig) {
    return lastResult;
  }
  lastVisibleTypes = visibleTypes;
  lastIconConfig = iconConfig;

  if (!iconConfig || iconConfig.length === 0) {
    lastResult = false;
    return false;
  }
  if (!visibleTypes || typeof visibleTypes.has !== 'function') {
    lastResult = false;
    return false;
  }

  let activityTypes = 0, selected = 0;
  for (const icon of iconConfig) {
    if (icon.enabled === false || !icon.activity_fallbacks) continue;
    activityTypes++;
    if (visibleTypes.has(icon.name)) selected++;
  }
  lastResult = selected > 0 && selected < activityTypes;
  return lastResult;
}

/**
 * Whether a trail (linear feature) should render given the current activity
 * narrowing. Untagged trails always show, so a missing tag never silently hides a
 * trail. (Selecting "Biking" hides hiking-only trails.)
 */
export function trailPassesActivityFilter(feature, visibleTypes, iconConfig) {
  if (!feature) return true;
  if (!isActivityFilterActive(visibleTypes, iconConfig)) return true;
  if (!(feature.primary_activities || '').trim()) return true;
  return poiMatchesActivityForTypes(feature, visibleTypes, iconConfig);
}


export function getIconUrlForPOI(poi, iconConfig, poiType) {
if (poiType === 'trail') return '/icons/layers/trails.svg';
if (poiType === 'river') return '/icons/layers/rivers.svg';
Expand Down