Skip to content

cleanup: remove stale wrappers and align progress reporting#23

Merged
blackopsrepl merged 1 commit intomainfrom
issue/10-cleanup-progress-model
Mar 21, 2026
Merged

cleanup: remove stale wrappers and align progress reporting#23
blackopsrepl merged 1 commit intomainfrom
issue/10-cleanup-progress-model

Conversation

@blackopsrepl
Copy link
Copy Markdown
Contributor

Summary

The progress API still exposed stale surface area: load_or_fetch_simple was an unused hidden wrapper, RoutingProgress still advertised an encoding phase with no producer, and matrix progress only emitted a single terminal update.

This change tightens that surface:

  • removes the unused load_or_fetch_simple wrapper
  • trims the dead EncodingGeometries progress variant
  • emits Complete consistently from fetch, matrix, and geometry operations
  • upgrades matrix progress to emit incremental row completion updates while keeping row computation parallel
  • updates the README progress example and variant table to match the actual model

Validation

  • cargo fmt --all -- --check
  • cargo clippy --workspace --all-targets -- -D warnings
  • cargo test

Fixes #10

@blackopsrepl blackopsrepl merged commit c998170 into main Mar 21, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

cleanup: remove stale wrappers and complete the progress event model

1 participant