Andrew Pendleton
Andrew Pendleton
I don't have review powers in this repo, but my two cents: one result of this change is that the rhumb-line intermediate and intermediate-fill operations might now panic given some...
For sure, agreed that that's the circumstance where it matters. I think implementation isn't that important (do we check up front for -90 < x < 90 or do we...
@joe-saronic sorry for the delay. Yes, I think you make a good case that clamping doesn't make sense here. > I think that refusing to do the computation is totally...
For reference, here's a quick port of this test program to Javascript so I could test it with Turf (which is what the Rust implementation was ported from): ```Javascript const...
An `Option` seems reasonable to me too, though that'd be a breaking API change, and would make rhumb destination inconsistent with haversine destination in terms of signature, so I could...
Makes sense. I'll probably have time to dig in in more detail either Friday or this weekend, but happy to cede this to someone else if there's interest in it...
@sebastian-hatala-leanix it looks like this is fixed in master but not the 3.x branch. What's the procedure for tracking that?
@springmeyer out of curiosity do you see writing code that handles different endianness properly to be part of this discussion, or a different issue?
A few more: * Sweden is `se` but Swedish is `sv` (and complicating things further, `sv` is also a country code, but it's El Salvador, and `se` is a valid...