helium / oracles

Oracles for Helium subDAOs
Apache License 2.0
17 stars 18 forks source link

Landtype Boosting #789

Closed michaeldjeffrey closed 2 months ago

michaeldjeffrey commented 2 months ago

https://github.com/helium/proto/pull/398

Add Landtype to Hex Boosting.


struct HexAssignments

Assignments are never considered individually, nor should they be. This type packs them together, allowing any code that requires specific use of Assignment order to attach itself here.

It also allows boostring_oracles/mod.rs to reduce it's reliance on traits. There is now a single exported BoostedHexAssignments that allows for getting a HexAssignments.


HexBoostData and CoverageDaemon

With only exporting a single trait from boosting_oracles/mod.rs, the CoverageDaemon now takes the concrete struct HexBoostData with no generic arguments.

Downstream from the CoverageDaemon, functions will take impl BoostedHexAssignments so they can be used in tests.


verify-disktree --path <PATH> --type <TYPE> CLI Command

A CLI command was added to mobile-verifier. It verifies a .h3tree file by making sure that every cell can be turned into an Assignment.

Current only Landtype is supported.

NOTE: In release mode, this function took about 8 minutes to run on my 2.3GHz Intel MacBook Pro for a 28gb file.

cargo run --release --features "file-store/local" -p mobile-verifier -- -c <CONFIG> verify-disktree --help

Other