Skip to content

feat: Overhaul Fly-Along with Nav Waypoints and Supersonic Pacing#28

Open
dkhawk wants to merge 4 commits intomainfrom
feat/route-sample
Open

feat: Overhaul Fly-Along with Nav Waypoints and Supersonic Pacing#28
dkhawk wants to merge 4 commits intomainfrom
feat/route-sample

Conversation

@dkhawk
Copy link
Collaborator

@dkhawk dkhawk commented Mar 5, 2026

Summary

This PR overhauls the cinematic Fly-Along feature for the Route Demo.

Key Changes

  • Navigation Waypoints: Camera now follows actual RouteStep locations from the Routes API.
  • Curve Following: Injected intermediate polyline points into long segments (>500m) to follow highway curves realistically.
  • Supersonic Calibration: Target speed set to 750m/s (Mach 2.2) at 3000m altitude.
  • Smart Merging: Skips waypoints closer than 200m to eliminate urban stutter.
  • Raw Visualization: Reverted to raw API polyline for exact flight path parity.
  • Variable Time Logic: Consistent perceived speed using the Haversine distance formula with snappy [250ms, 2s] clamping.

Verification

  • Build successful: ./gradlew :Maps3DSamples:advanced:app:compileDebugKotlin
  • Manual verification performed on long highway and short city routes.

@dkhawk dkhawk requested a review from kikoso March 5, 2026 23:46
@dkhawk dkhawk force-pushed the feat/route-sample branch from 45573fb to ce0e688 Compare March 5, 2026 23:51
@@ -0,0 +1,104 @@
// Copyright 2025 Google LLC
Copy link
Collaborator

Choose a reason for hiding this comment

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

We don't have the header check in this repo, but being a new file, it could be worth adding the 2026 year.

@@ -0,0 +1,80 @@
// Copyright 2025 Google LLC
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
// Copyright 2025 Google LLC
// Copyright 2026 Google LLC

@@ -0,0 +1,104 @@
// Copyright 2025 Google LLC
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
// Copyright 2025 Google LLC
// Copyright 2026 Google LLC

@@ -0,0 +1,335 @@
// Copyright 2025 Google LLC
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
// Copyright 2025 Google LLC
// Copyright 2026 Google LLC

})
},
onRelease = { view ->
// Optional cleanup
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do we need any release method on the map? (more of an open question)

Copy link
Collaborator

@kikoso kikoso left a comment

Choose a reason for hiding this comment

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

LGTM! Left a couple of minor comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants