Skip to content

fix(examples): use bundled Scenegraph flight snapshot - #10476

Open
ibgreen-openai wants to merge 2 commits into
masterfrom
codex/scenegraph-example-local-data
Open

fix(examples): use bundled Scenegraph flight snapshot#10476
ibgreen-openai wants to merge 2 commits into
masterfrom
codex/scenegraph-example-local-data

Conversation

@ibgreen-openai

@ibgreen-openai ibgreen-openai commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

Goal

opensky URLs now seem to fail due to CORS restructions.
Make the Scenegraph website example load reliably even when the OpenSky browser API is unavailable.

Changes

  • Start from the bundled all.json flight snapshot so the example renders immediately.
  • Try the live OpenSky endpoint once, then keep the bundled snapshot if the request is blocked by CORS or otherwise fails.
  • Normalize data without mutating the imported snapshot.
  • Remove the interval/update loop that could create repeated fetch failures.
  • Rename the demo from “Realtime Flight Tracker” to “Flight Tracker” and document the fallback behavior.

This PR intentionally contains only example/data-source behavior. The ScenegraphLayer WebGPU port is split into a separate stacked PR.

Validation

  • yarn lint (passes with existing repository warnings only)
  • yarn test-website

@coveralls

coveralls commented Jul 23, 2026

Copy link
Copy Markdown

Coverage Status

coverage: 83.123% (-0.005%) from 83.128% — codex/scenegraph-example-local-data into master

@ibgreen-openai
ibgreen-openai marked this pull request as ready for review July 23, 2026 22:12
@ibgreen-openai
ibgreen-openai force-pushed the codex/scenegraph-example-local-data branch from ba6afdf to 44751f7 Compare July 27, 2026 20:22
@ibgreen
ibgreen force-pushed the codex/scenegraph-example-local-data branch from 80b959e to 5bac2f5 Compare August 1, 2026 18:04
@ibgreen ibgreen added this to the v9.4 milestone Aug 2, 2026
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.

3 participants