ava-labs / ava-labs/reforge

Change OffsetAdjustment to be a BTreeMap

Open
#27 0 comments 0 reactions 0 assignees View on GitHub
enhancement good first issue rust
Dominant language
Rust
Stars
0
Forks
0
Avg merge
10d 6h
Merged PRs (30d)
4

Description

A performance improvement over code which uses a vector and many filter maps. It also means that t Deref/DerefMut impl on OffsetAjustment should be removed.

Contributor guide

Open the contributing guide

Research direction

Locate the OffsetAdjustment definition and its current vector, filter-map usage, and Deref/DerefMut implementations. Compare the existing behavior before changing the representation to a BTreeMap, then verify that the conversion is complete and both dereference implementations are removed.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
tooling
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
58/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.