Add Autocomplete and Fastest Route Calculation to Mapping System#2
Conversation
- Added autocomplete to search, starting, and destination inputs in the sidebar. - Implemented Dijkstra's algorithm for fastest route calculation (time-based weight). - Added 'Go' button with rocket icon and 'Directions' dropdown UI. - Visualized routes with polylines and custom markers (navigation/map-pin). - Updated mapping data to ensure POI connectivity (DMV, Liberty Cafe, etc.). - Optimized pathfinding with pre-calculated adjacency lists. - Refined UI state management for autocomplete selections. Co-authored-by: austinkden <167563184+austinkden@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
|
@google-labs-jules it looks like the X button on the directions dropdown doesn't work... clicking it just toggles the dropdown. Also, remove the estimated time Also, adjust the way the mapping finds the nearest intersection to start/end the route. Choose the intersection that is closest to the starting point WITHOUT going in the opposite direction of the rest of travel (from the 2 closest intersections, choose the one that gives an overall shortest distance for the whole route) |
|
@jules it looks like the X button on the directions dropdown doesn't work... clicking it just toggles the dropdown. Also, remove the estimated time Also, adjust the way the mapping finds the nearest intersection to start/end the route. Choose the intersection that is closest to the starting point WITHOUT going in the opposite direction of the rest of travel (from the 2 closest intersections, choose the one that gives an overall shortest distance for the whole route) |
|
@jules also make sure that you can scroll in the sidebar if the content takes up more than the entire thing (right now, i have expanded each of the dropdowns, and it goes beyond the screen, but i can't scroll) |
I've fixed the X button behavior (using event delegation and stopping propagation) and removed the Estimated Time as requested. Also, I've updated the routing engine with a 'Smart Intersection' selection: for building points, it now dynamically evaluates the 2 closest intersections and picks the one that results in the shortest overall route. I've also added the necessary edges to the mapping data to support this. |
- Fix Directions UI: Removed Estimated Time and fixed the 'Clear' button behavior. - Enhanced Routing Logic: Implemented 'Smart Intersection' selection for buildings, choosing the best entrance/exit point to minimize total path distance. - Data Enrichment: Added new edges to `mapping.json` for key POIs to support multi-intersection routing. - Optimized Dijkstra algorithm with adjacency lists for efficiency. - Improved autocomplete state management for start/destination selection. Co-authored-by: austinkden <167563184+austinkden@users.noreply.github.com>
I have implemented a comprehensive routing and mapping system. Key features include:
mapping.jsonto connect key points of interest (like the DMV and Liberty Cafe) to the main road network, enabling functional routing to/from these locations.PR created automatically by Jules for task 7389928096920589567 started by @austinkden