facebookresearch / facebookresearch/sam3
Is look-forward-twice (Zhang et al., 2022a) actually used? (as the paper states)
- Dominant language
- Python
- Stars
- 11.7k
- Forks
- 1.8k
- PR merge metrics
- No merged PRs in 30d
Description
The paper mentions `"Following standard practice in stronger DETR variants, we use [...]
look-forward-twice (Zhang et al., 2022a) [...]"`
However the implementation at https://github.com/facebookresearch/sam3/blob/5eb25fb54b70ec0cb16f949289053be091e16705/sam3/model/decoder.py#L555-L573
looks to me like it is the standard look once regime? Am I missing something?
Contributor guide
Research direction
Start at sam3/model/decoder.py lines 555-573 and compare the implementation there with the paper's statement about look-forward-twice (Zhang et al., 2022a). Determine whether the code implements the stated regime or standard look-once behavior, and document the evidence and any discrepancy.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- computer-vision
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 42/100