Skip to content

clean up index helpers and repo-level md files - #30

Merged
kylecarow merged 2 commits into
mainfrom
index-helpers
Aug 7, 2026
Merged

clean up index helpers and repo-level md files#30
kylecarow merged 2 commits into
mainfrom
index-helpers

Conversation

@kylecarow

@kylecarow kylecarow commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Splits src/strategy/traits.rs into trait definitions (unchanged) and a new
    src/strategy/utils.rs holding the grid/index search helpers those traits'
    interpolate impls call into, matching the split the in-progress cubic-spline
    branch (CubicSpline #16) already landed independently.
  • Renames find_nearest_indexlocate_lower_index (public, no deprecation shim;
    pre-1.0, no stability guarantee yet), step_indexlocate_step_index,
    uniform_lower_indexlocate_lower_index_uniform.
  • Folds the out-of-range clamp into locate_lower_index itself instead of leaving it
    hand-inlined at every Linear call site. A first draft of Multi-channel ("shared-grid") interpolation support & DynInterpolator/DynInterpolatorMulti #19's multi-channel
    Linear override had reinvented this search without the clamp and panicked on
    extrapolated points; this closes that gap for good.
  • Adds AxisLocation<T> + locate_axis, and uses them to replace the duplicated
    clamp-array/exact_index logic in Linear's 1D/2D/3D interpolate (the ND impl
    keeps its own dimensionality-reduction logic, just drops its now-redundant manual
    clamp).
  • Renames CHANGELOGCHANGELOG.md and LICENSELICENSE.md so GitHub renders
    them instead of showing raw Markdown as plain text.

Test plan

  • cargo build --all-features
  • cargo test --all-features (68 unit tests, 15 doc tests, all passing)
  • cargo clippy --all-features --all-targets (no new warnings vs. main)
  • cargo fmt --check

@kylecarow
kylecarow merged commit cbd958d into main Aug 7, 2026
1 check passed
@kylecarow
kylecarow deleted the index-helpers branch August 7, 2026 02:41
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.

1 participant