Skip to content

Conversation

@ocrete
Copy link
Collaborator

@ocrete ocrete commented Nov 20, 2025

No description provided.

@ocrete ocrete requested a review from dmorin1 November 20, 2025 22:42
Comment on lines +40 to +41
bounding-box-center-x = top-left-x * CENTER_VARIANCE * anchor-width + anchor-center-x
bounding-box-center-y = top-left-y * CENTER_VARIANCE * anchor-height + anchor-center-y
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't really understand this formula but maybe once you clarify anchor-center-x and anchor-center-y. It will be more clear.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The formula is taken from the tensor decoder... It moves the anchor box by a certain value I think

```
bounding-box-center-x = top-left-x * CENTER_VARIANCE * anchor-width + anchor-center-x
bounding-box-center-y = top-left-y * CENTER_VARIANCE * anchor-height + anchor-center-y
bounding-box-width = expf (width * SIZE_VARIANCE) * anchor-width
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is expf ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's expf(x) is the C function that does e^(x) .. would that be clearer ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

something like:

Suggested change
bounding-box-width = expf (width * SIZE_VARIANCE) * anchor-width
bounding-box-width = e^(width * SIZE_VARIANCE) * anchor-width

@ocrete ocrete force-pushed the ultra-light-face-detector-without-postproc branch from f6a1fae to 581b090 Compare November 24, 2025 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants