facebookresearch / facebookresearch/sam3

Is look-forward-twice (Zhang et al., 2022a) actually used? (as the paper states)

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

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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.