Skip to content

fix linear projection for bounding box binary classification#148

Open
NouamaneTazi wants to merge 1 commit intofundamentalvision:mainfrom
NouamaneTazi:patch-1
Open

fix linear projection for bounding box binary classification#148
NouamaneTazi wants to merge 1 commit intofundamentalvision:mainfrom
NouamaneTazi:patch-1

Conversation

@NouamaneTazi
Copy link

@NouamaneTazi NouamaneTazi commented May 22, 2022

The paper says:

The detection head is of a 3-layer FFN for bounding box regression, and a linear projection for bounding box binary classification (i.e., foreground and background)

Doesn't that mean we should only have 2 outputs in class_embed? (later used in here)

EDIT: after further investigation it seems that my confusion comes from this line . Why do we pick the best scoring bounding boxes based on the first class?

The paper says:
| The detection head is of a 3-layer FFN for bounding box regression, and a linear projection for bounding box binary classification (i.e., foreground and background)
doesn't that mean we should only have 2 outputs in the class_embed (later used in here)
@NouamaneTazi
Copy link
Author

Related to #79

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant