uber / h3

Hexagonal hierarchical geospatial indexing system
https://h3geo.org
Apache License 2.0
4.95k stars 470 forks source link

All remaining subcommands for the h3 cli program #924

Closed dfellis closed 1 month ago

dfellis commented 1 month ago

After implementing the region subcommands, all of the rest documented on h3geo.org were "trivial" as all patterns of subcommand had been implemented, so I just slogged through writing all of them while I had the drive to do it. ;)

There may be some test failures with degsToRads and friends due to floating point rounding errors on different CPU architectures. I hope not, but if so, then I'll have to figure out how to test them better.

dfellis commented 1 month ago

Oh, this currently includes the region subcommands in this PR, since it depends on that one, but once it gets approved and merged, this'll be reduced to only the new commands.

I did (almost) every command in a separate commit, for easier reviewing.

coveralls commented 1 month ago

Coverage Status

coverage: 98.782%. remained the same when pulling d63accfd1d5c164b339fb38d5643f16d1d171295 on h3-cli-remaining-subcommands into 33681ff65bc4bb4a3addf8dbef5c6b070f811b27 on master.

dfellis commented 1 month ago

Discussed offline, some of the inconsistencies will be taken care of in a follow-up PR.