Skip to content

refactor(cesium): six-page rebuild around programmatic data models - #65

Closed
parker-super wants to merge 3 commits into
masterfrom
refactor/cesium-six-pages
Closed

parker-super wants to merge 3 commits into
masterfrom
refactor/cesium-six-pages

Conversation

@parker-super

Copy link
Copy Markdown
Collaborator

Drops all static JSON inputs. Shared viewer restores sky/atmosphere with a cinematic entry flyover and an effects kit; GeoHash aggregates into extruded columns; Situation tracks billboard targets with lock-follow; Direction/Heatmap/Trajectory fully rewritten (antenna-pattern capability rings with flat-only geometry per the engine's extrusion crash matrix, canvas-texture heat field and sweep, Catmull-Rom patrol replay with a growing corridor). Smoke tests pin value ranges.

… shared effects

Shared viewer keeps Cesium's sky/atmosphere on, deep-blue globe fallback color, ground atmosphere, an entry flyover (high-altitude setView then 2.2s eased flyTo) with instant mode for deterministic tests, and an effects kit (scan rings, glow material, target icons) reused by the pages.
…et tracking

GeoHash aggregates on entry and recalculates on precision change with extruded columns encoding density; Situation gains billboard targets with heading, sea wall, target lock via tracked entity and a following scan ring.
…ogrammatic models

All static JSON inputs dropped. Direction: antenna-pattern capability rings (flat clamped geometry only - the engine nondeterministically crashes workers on large extruded polygons), rotating sweep via canvas afterglow texture, orbiting targets with live intercept state; camera frames the rings top-down on entry. Heatmap: propagation-model field re-rendered to a canvas texture with coverage shells and transmitter scan rings. Trajectory: Catmull-Rom patrol replay with a corridor that grows with progress and progressive link-quality samples. Smoke tests pin sane value ranges (e.g. capability distances 60-320 km).
@coderabbitai

coderabbitai Bot commented Oct 18, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 59 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 36112dd2-68a0-477b-92e4-9c0637f2035e
📥 Commits

Reviewing files that changed from the base of the PR and between 66d9b6a and b25a5a8.

📒 Files selected for processing: 14
  • src/components/CesiumViewer/effects.ts
  • src/components/CesiumViewer/index.tsx
  • src/components/CesiumViewer/lifecycle.test.tsx
  • src/locales/en-US/map.ts
  • src/locales/zh-CN/map.ts
  • src/pages/Map/Cesium/Direction.tsx
  • src/pages/Map/Cesium/GeoHash.tsx
  • src/pages/Map/Cesium/Heatmap.tsx
  • src/pages/Map/Cesium/Situation.tsx
  • src/pages/Map/Cesium/Trajectory.tsx
  • src/pages/Map/Cesium/capabilityModel.test.ts
  • src/pages/Map/Cesium/capabilityModel.ts
  • src/pages/Map/Cesium/fieldModel.ts
  • src/pages/Map/Cesium/patrolModel.ts
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying umi-react-admin with  Cloudflare Pages  Cloudflare Pages

Latest commit: b25a5a8
Status:🚫  Build failed.

View logs

@Jerry-CodeHub

Copy link
Copy Markdown
Owner

改由仓库账号 Jerry-CodeHub 重新发起(PR 作者需与仓库归属一致)。

@Jerry-CodeHub
Jerry-CodeHub deleted the refactor/cesium-six-pages branch October 18, 2026 03:37
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