Skip to content

Asterisk imports in __init__.py submodules? #645

Description

@Icemole

In #643 @albertz noticed that in the submodule's __init__.py files we imported everything from the submodule itself with an asterisk import. For instance:

  1. AM module
  2. Audio module
  3. LM module
  4. RETURNN module

Do we want to allow this or not? I think using asterisk imports in __init__.py has upsides (more aesthetic/sensible grouping of jobs, especially when boundaries between py files are not too well defined) and downsides (many of these imports are not need at manager loading time, so the final loading time is increased).

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions