google-research / t5x

Apache License 2.0
2.67k stars 306 forks source link

Missing `__init__.py` in `t5x/t5x/examples/decoder_only` #1015

Open rpeloff opened 1 year ago

rpeloff commented 1 year ago

The directory t5x/t5x/examples/decoder_only is missing an __init__.py file. This means that the modules (decoder-only layers and network) cannot be imported in gin configurations.

coreyfournier commented 1 year ago

What version of python are you using? 3.3+ don't require it.

https://stackoverflow.com/questions/448271/what-is-init-py-for

sinoroc commented 1 year ago

As far as I know, __init__.py should be in every importable package except namespace packages. Even though it is true that in some (many) cases everything will work as expected without such a file. If there is no __init__.py, then technically it is considered a "namespace package", and it does have some subtle differences.

References: