jhjacobsen / invertible-resnet

Official Code for Invertible Residual Networks
MIT License
517 stars 78 forks source link

Undefined names: Missing imports? #1

Closed cclauss closed 5 years ago

cclauss commented 5 years ago

flake8 testing of https://github.com/jhjacobsen/invertible-resnet on Python 3.7.1

$ flake8 . --count --select=E9,F63,F72,F82 --show-source --statistics

./CIFAR_main.py:311:102: F821 undefined name 'full_fname'
        interpolate(model, testloader, testset, start_epoch, use_cuda, best_objective, args.dataset, full_fname)
                                                                                                     ^
./models/utils_toy_densities.py:250:19: F821 undefined name 'model'
    clip_fc_layer(model, coeff, use_cuda)
                  ^
./models/utils_toy_densities.py:250:26: F821 undefined name 'coeff'
    clip_fc_layer(model, coeff, use_cuda)
                         ^
./models/utils_toy_densities.py:250:33: F821 undefined name 'use_cuda'
    clip_fc_layer(model, coeff, use_cuda)
                                ^
./models/utils_toy_densities.py:257:41: F821 undefined name 'model'
    out_bij, p_z_g_y, trace, gt_trace = model(inputs)
                                        ^
./models/utils_toy_densities.py:258:31: F821 undefined name 'model'
    log_det = compute_log_det(model, inputs, out_bij)
                              ^
./models/utils_toy_densities.py:277:41: F821 undefined name 'model'
    out_bij, p_z_g_y, trace, gt_trace = model(inputs)
                                        ^
./models/utils_toy_densities.py:278:31: F821 undefined name 'model'
    log_det = compute_log_det(model, inputs, out_bij)
                              ^
./models/model_utils.py:226:76: F821 undefined name 'num_units'
                         'multiple of group_size({})'.format(num_channels, num_units))
                                                                           ^
./models/invertible_layers.py:181:26: F821 undefined name 'Conv2dZeroInit'
        self.conv_zero = Conv2dZeroInit(c // 2, c, 3, padding=(3 - 1) // 2)
                         ^
./models/invertible_layers.py:187:16: F821 undefined name 'gaussian_diag'
        return gaussian_diag(mean, logs)
               ^
./models/invertible_layers.py:215:21: F821 undefined name 'NN_actnorm'
          self.NN = NN_actnorm(H, W, in_channels=num_features // 2, hidden_channels=width)
                    ^
./models/invertible_layers.py:217:21: F821 undefined name 'NN_layernorm'
          self.NN = NN_layernorm(H, W, in_channels=num_features // 2, hidden_channels=width)
                    ^
./models/invertible_layers.py:219:21: F821 undefined name 'NN_batchnorm'
          self.NN = NN_batchnorm(H, W, in_channels=num_features // 2, hidden_channels=width)
                    ^
./models/invertible_layers.py:237:21: F821 undefined name 'NN_actnorm'
          self.NN = NN_actnorm(H, W, in_channels=num_features // 2, hidden_channels=width, channels_out=num_features)
                    ^
./models/invertible_layers.py:239:21: F821 undefined name 'NN_layernorm'
          self.NN = NN_layernorm(H, W, in_channels=num_features // 2, hidden_channels=width, channels_out=num_features)
                    ^
./models/invertible_layers.py:241:21: F821 undefined name 'NN_batchnorm'
          self.NN = NN_batchnorm(H, W, in_channels=num_features // 2, hidden_channels=width, channels_out=num_features)
                    ^
./models/invertible_layers.py:250:22: F821 undefined name 'flatten_sum'
        objective += flatten_sum(torch.log(scale))
                     ^
./models/invertible_layers.py:261:22: F821 undefined name 'flatten_sum'
        objective -= flatten_sum(torch.log(scale))
                     ^
19    F821 undefined name 'full_fname'
19

E901,E999,F821,F822,F823 are the "showstopper" flake8 issues that can halt the runtime with a SyntaxError, NameError, etc. These 5 are different from most other flake8 issues which are merely "style violations" -- useful for readability but they do not effect runtime safety.