Repository navigation
fix(satellites): refuse invalid observer coordinates before look-angle propagation - #1050
Closed
rudycelekli wants to merge 1 commit into
Conversation
rudycelekli
requested review from
bilawalsidhu and
samehkhamis
as code owners
October 11, 2026 14:44
Collaborator
|
Thanks @rudycelekli for checking the look-angle helper against bad observer input; the boundary tests are clearly written. I am going to close this one because the helper never sees that input on main. gods-eye-view/src/data/satellitePass.js Lines 211 to 235 in 591f299 satellites_overhead tool gets its lat and lon through the point schema (gods-eye-view/src/tools/area.js Lines 109 to 118 in 591f299 run (gods-eye-view/src/tools/catalog.js Line 181 in 591f299 lookAnglesAt would repeat them on every step of the pass search without changing any result.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The public look-angle helper accepted impossible or absent observer coordinates, producing plausible angles for out-of-range locations or returning NaN angle objects.
Validate finite numeric observer coordinates, WGS84 ranges and a representable UTC instant before propagation; retain null as the helper’s unavailable outcome.
Validation: native regression fails on unchanged main and passes with the change, along with the adjacent source tests. Changed JavaScript passes Prettier and the diff whitespace check.
The combined compatibility branch passes the full unit suite (6,274 passed, one skipped), production build, package boundaries, allocation gates and browser tracking (109 passed). Browser checks use isolated local imagery and terrain fixtures; they do not establish live-source acceptance or geographic measurement accuracy.