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

Problem about ComplexBN #19

Open
jackie840129 opened this issue Jan 11, 2018 · 1 comment
Open

Problem about ComplexBN #19

jackie840129 opened this issue Jan 11, 2018 · 1 comment

Comments

@jackie840129
Copy link

jackie840129 commented Jan 11, 2018

Hi,
I have some question about this ComplexBN class at line 409,410,

if training in {0, False}:
            return input_bn

why we don't need moving_mean and moving_variance when testing?

I also see another function at line 421,

def normalize_inference():
                if self.center:
                    inference_centred = inputs - K.reshape(self.moving_mean, broadcast_mu_shape)
                else:
                    inference_centred = inputs
                return ComplexBN(
                    inference_centred, self.moving_Vrr, self.moving_Vii,
                    self.moving_Vri, self.beta, self.gamma_rr, self.gamma_ri,
                    self.gamma_ii, self.scale, self.center, axis=self.axis
                )

When testing, this function doesn't be called. Is there any mistake?

The code below are the source code in Keras, they return "normalize_inference()" when testing.

 if training in {0, False}:
           return normalize_inference()

Thanks!

@DevKiHyun
Copy link

I think so. Why use not normalize_inference when test?

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