facebookresearch / facebookresearch/theseus
Change VariableOrdering so that indexing happens using integers rather than strings.
Open
performance
- Dominant language
- Python
- Stars
- 2.1k
- Forks
- 149
- PR merge metrics
- No merged PRs in 30d
Description
## 🚀 Feature
Current variable indexing uses string, which impacts efficiency.
## Motivation
Switching to integers under the hood may result in substantial speed up.
Contributor guide
Research direction
Start by locating the VariableOrdering implementation and its existing tests or benchmarks. Trace how variables are currently indexed, then define completion as preserving current behavior while changing the internal indexing representation from strings to integers and verifying the efficiency impact.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- tooling
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 30/100