Got an error very recently for "layers.normalizations". Something related to the "epsilon" variable in the typeguard module.

[davidweioct]

The code was run in google colab with the following install libraries:

!pip3 install nilearn
!pip3 install simpleITK
!pip3 install tensorflow-gpu==2.1.0
!pip3 install tensorflow-addons==0.8.1

ERROR info:

File "/usr/local/lib/python3.6/dist-packages/tensorflow_addons/layers/normalizations.py", line 319, in init
super().init(**kwargs)
File "/usr/local/lib/python3.6/dist-packages/typeguard/init.py", line 809, in wrapper
check_argument_types(memo)
File "/usr/local/lib/python3.6/dist-packages/typeguard/init.py", line 670, in check_argument_types
raise exc from None
File "/usr/local/lib/python3.6/dist-packages/typeguard/init.py", line 668, in check_argument_types
check_type(description, value, expected_type, memo)
File "/usr/local/lib/python3.6/dist-packages/typeguard/init.py", line 598, in check_type
format(argname, qualified_name(expected_type), qualified_name(value)))
TypeError: type of argument "epsilon" must be int; got float instead

Thank you so much!

[davidweioct]

It was running good previously. Not sure if this is due to a recent update.

[davidweioct]

I used a work-around solution by simply downgrading to 0.7.1

[seanpmorgan]

This is a good catch @davidweioct
https://github.com/tensorflow/addons/blob/master/tensorflow_addons/layers/normalizations.py#L50
https://www.tensorflow.org/api_docs/python/tf/nn/batch_normalization

Should be a float not integer.

[failure-to-thrive]

So isn't it just a change of int to float?

addons/tensorflow_addons/layers/normalizations.py

Line 78 in 343e7f3

epsilon: int = 1e-3,

What's the catch?

[seanpmorgan]

So isn't it just a change of int to float?

addons/tensorflow_addons/layers/normalizations.py

Line 78 in 343e7f3

epsilon: int = 1e-3,

What's the catch?

The catch is that we incorrectly typed it as an int. If we have casting then we can make it Union[int, float], but otherwise it should be typed as float

[failure-to-thrive]

I mean, from a workflow point of view. It's a very simple (even "stylish") modification that could be done in no time without a contributors' help. Sorry, I'm just unfamiliar with practices of this project.

[seanpmorgan]

I mean, from a workflow point of view. It's a very simple (even "stylish") modification that could be done in no time without a contributors' help. Sorry, I'm just unfamiliar with practices of this project.

Addons only consists of volunteer contributors 😄

Please feel free to submit a PR, as you mentioned its an easy fix... and maintainers need to balance reviews, shepparding issues (like this one) and submitting contirbutions....

[AakashKumarNain]

eps should be float only and not int in any case.