Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

facial-landmarks-35-adas-0001 input description #47

Closed
banderlog opened this issue Mar 15, 2019 · 5 comments
Closed

facial-landmarks-35-adas-0001 input description #47

banderlog opened this issue Mar 15, 2019 · 5 comments

Comments

@banderlog
Copy link
Contributor

It stated:

The blob is constructed from a 60x60 pixel BGR image with subtracted mean of (120, 110, 104) and scale factor of 0.0039.

And I have spent a lot of time, to figure out, that right parameters must come without mean and scale, or you'll get warped/distorted result.

# original, not working
blob = cv2.dnn.blobFromImage(face, 0.0039, (60, 60), mean=[120, 110, 104], swapRB=False)

# working
blob = cv2.dnn.blobFromImage(face, 1, (60, 60))
# also working, but worser
blob = cv2.dnn.blobFromImage(face, 1, (60, 60), mean=[120,110,104], swapRB=False)

@snosov1
Copy link
Contributor

snosov1 commented Mar 18, 2019

Hey, @banderlog !

Sorry, I'm having hard time to extract an actionable item from your description.

  • Do you imply that the documentation is misleading (e.g. simply eliminating the mean/scale text from the document will avoid the confusion)?
  • Is there an actual code in this repo that needs to be changed?

@banderlog
Copy link
Contributor Author

My bad, @snosov1
Yes, I'm implying that the documentation is misleading and simple 1 string change will help to avoid confusion.

@snosov1
Copy link
Contributor

snosov1 commented Mar 18, 2019

I'm with you on that one, care to submit a PR?

@snosov1
Copy link
Contributor

snosov1 commented Mar 18, 2019

Sorry, I see you've already created one #48 =)

@snosov1
Copy link
Contributor

snosov1 commented Mar 29, 2019

Not relevant anymore with 0002 replacing 0001 model

@snosov1 snosov1 closed this as completed Mar 29, 2019
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

No branches or pull requests

2 participants