Using 0-indexing for the tree indices
Open
refactoring
- Dominant language
- JavaScript
- Stars
- 56
- Forks
- 32
- PR merge metrics
- No merged PRs in 30d
Description
Suggestion from @dhakim87 during code review today. Indices are currently 1-indexed to resemble the original BP paper.
Contributor guide
Research direction
The issue does not name files, tests, or entry points. Trace how tree indices are assigned and consumed in the JavaScript visualization, then identify every affected display or lookup; done means the tree uses 0-based indices consistently and existing behavior remains covered by tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- data-visualization
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 42/100