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

Confusion of normalization setup for VNNCOMP networks #4

Open
JacksonZyy opened this issue Mar 2, 2023 · 0 comments
Open

Confusion of normalization setup for VNNCOMP networks #4

JacksonZyy opened this issue Mar 2, 2023 · 0 comments

Comments

@JacksonZyy
Copy link

Hi,

I am reading your documentation and config files to try out MN-BAB.
I noticed that you set up normalization as follows for cifar10_conv_small.json, which seems to be reasonable as the original onnx model from ERAN indeed includes the sub and div layers.

  "normalization_means": [
    0.4914,
    0.4822,
    0.4465
  ],

But I wonder why you setup the same normalization for configs/cifar10_resnet_3b2_bn.json? I noticed that this network is from VNNCOMP and it doesn't contain normalization layer like sub/div. Then isn't it wrong to add this normalization as it changes the computation?
I would be grateful if you could provide some insight of this problem.
Thank you!

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

1 participant