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

confuse about some code. #52

Open
tpys opened this issue May 7, 2018 · 4 comments
Open

confuse about some code. #52

tpys opened this issue May 7, 2018 · 4 comments

Comments

@tpys
Copy link

tpys commented May 7, 2018

There are some code that I can't figure out by myself, please help!

  1. in generator.py file, should the true_box_index be set to zero for every train_instance ?

    true_box_index = 0

  2. in yolo.py file, pred_box_conf - 0 and true_box_wh + tf.zeros_like(true_box_wh) * (1-object_mask) make no difference, I really confused about warmup training, what it is trying to do ?
    please give more explanations.

    conf_delta = pred_box_conf - 0

true_box_wh + tf.zeros_like(true_box_wh) * (1-object_mask),

  1. in YoloLayer there is a parameter named max_grid, when create model you pass the max_input_size to it, which is 480. but inside the create_yolov3_model this value has been multiply by 2 or 4 in different yolo layer. is that right?
    [2*num for num in max_grid],

    [4*num for num in max_grid],

    Thanks a lot !!!
@experiencor
Copy link
Owner

  1. set to 0 every batch, not instance
  2. 👍, my bad
  3. yup, the grid size is doubled in size in later scales

@tpys
Copy link
Author

tpys commented May 9, 2018

@experiencor Thank you for kindly reply.

@muye5
Copy link

muye5 commented May 17, 2018

should 'max_grid ' be 480 or 480 / downsample (e.g. 480 / 32) . @experiencor @tpys

@artest08
Copy link

artest08 commented Oct 25, 2019

  1. set to 0 every batch, not instance
  2. +1, my bad
  3. yup, the grid size is doubled in size in later scales

For the second one, is there an update. In the code still the line
true_box_wh + tf.zeros_like(true_box_wh) * (1-object_mask).

What should be this line? Should we add the width and height of the anchor boxes there?

I think the change can be done in this way:
true_box_wh + tf.ones_like(true_box_wh) *
self.anchors / net_factor *
(1-object_mask),

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

4 participants