facebookresearch / facebookresearch/theseus

Change VariableOrdering so that indexing happens using integers rather than strings.

Open
#6 0 comments 0 reactions 0 assignees View on GitHub
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.