search

uber / h3

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

MAX_H3_RES should be tested inclusive #820

Closed isaacbrodsky closed 5 months ago

isaacbrodsky commented 5 months ago

I think this is an oversight in these tests, MAX_H3_RES is the last valid resolution, so we should include it.

coveralls commented 5 months ago

Coverage Status

coverage: 98.827%. remained the same when pulling 7d841a68657b59e71ef675ea1855a1b0df7ce1af on isaacbrodsky:max-h3-res-inclusive into 094699baf7fbaf6fca4697f6dc4ce78b5bcd1c38 on uber:master.