Skip to content

salvage(master): hopi spiral route generator + expanded tests - #114

Merged
THOClabs merged 1 commit into
mainfrom
salvage/hopi-spiral-routes
Aug 7, 2026
Merged

salvage(master): hopi spiral route generator + expanded tests#114
THOClabs merged 1 commit into
mainfrom
salvage/hopi-spiral-routes

Conversation

@THOClabs

@THOClabs THOClabs commented Aug 7, 2026

Copy link
Copy Markdown
Owner

Summary

Final salvage before the branch purge: master's 8 unique commits are otherwise superseded (NEO clients → #108/#109; event log/space weather → #111; dated March-2026 event-scan snapshots), but its spiral route generator for hopi_circles (176 additive lines) and the expanded hopi_circles/trajectory test suites are genuinely new. Purely additive; 156 affected tests pass.

After this merges, master is deleted along with the 86 script-targeted branches.

🤖 Generated with Claude Code


Note

Low Risk
Purely additive domain math and tests in meteor tracking; no API, auth, or data-store changes.

Overview
Adds ground-search navigation on top of existing Hopi concentric circles: labeled Waypoint / SearchRoute models, geodesic helpers (destination_point, initial_bearing), per-circle waypoint placement, and a center-outward spiral that offsets each ring’s start bearing so routes interleave instead of stacking rings.

generate_spiral_route is the main entry point (builds circles, then SearchPattern.generate_spiral_route), with Lexicon-style to_prayer_string output and total route mileage. The legacy generate_waypoints_on_circle tuple API is left in place.

New test_hopi_circles and test_trajectory suites cover geodesic math, spiral geometry, max-radius limits, and an integration path from calculate_trajectory debris centers into spiral routes.

Reviewed by Cursor Bugbot for commit 93f4fb7. Bugbot is set up for automated code reviews on this repo. Configure here.

…ry tests

Recovered from the legacy master branch before its deletion: a purely
additive spiral search-route generator (Waypoint, SearchRoute,
generate_spiral_route, destination_point, initial_bearing) for
hopi_circles, plus the comprehensive test suites for hopi_circles and
trajectory (the only content of master's 8 unique commits not already
superseded by #108/#109/#111). All tests pass against current main.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 93f4fb7. Configure here.

math.cos(d_over_R) - math.sin(lat1) * math.sin(lat2)
)

return (math.degrees(lat2), math.degrees(lon2))

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Longitude not normalized after destination

Medium Severity

destination_point returns math.degrees(lon2) without wrapping longitude to the conventional ±180° range. Near the antimeridian, consecutive circle waypoints can mix values above 180° with negative longitudes, so haversine_miles segment and route totals in generate_spiral_route and SearchRoute.segment_distances can be badly wrong for otherwise valid search centers.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 93f4fb7. Configure here.

@THOClabs
THOClabs merged commit a5f0870 into main Aug 7, 2026
10 checks passed
@THOClabs
THOClabs deleted the salvage/hopi-spiral-routes branch August 7, 2026 04:22
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.

1 participant