Manifest refactor

[stasbel]

Old usage:

manifest = ManifestBase(  # Or `ManifestEN`.
    *args,
    **kwargs,
)

New usage:

# More conventional full path imports with namespaces to import from.
from nemo_asr.parts import collections
from nemo_asr.parts import parsers


# Make parser.
parser = parsers.make_parser(labels, 'en', **kwargs)
parser = parsers.ENCharParser(labels, **kwargs)  # Or.

# Make collection.
collection = collections.Text(texts, parser=parser)
collection = collections.FromFileText(file, parser=parser)  # Or.

# Use fields. Each example is namedtuple.
# Look for `collections.Class.OUTPUT_TYPE` for fields information.
print(collection[0].tokens)   # List of ints.

[lgtm-com]

This pull request introduces 1 alert and fixes 3 when merging 00a9d59 into bc0d7b1 - view on LGTM.com

new alerts:

• 1 for Unused import

fixed alerts:

• 3 for Unused import

[blisc]

Why did you request a review for a WIP PR?

[blisc]

The new classes inside manifests.py are not very intuitive to me. It's hard for me to understand what you are trying to do with them at first glance.

[stasbel]

Why did you request a review for a WIP PR?

So you can comment on draft. Now I know what's need to fixed first.

[stasbel]

Sorry, I have to force-push commits to re-sign them after rebasing.

[lgtm-com]

This pull request fixes 4 alerts when merging 8a22b19 into 7779728 - view on LGTM.com

fixed alerts:

• 3 for Unused import
• 1 for Except block handles 'BaseException'

[lgtm-com]

This pull request introduces 1 alert and fixes 4 when merging 59ba6c9 into 7779728 - view on LGTM.com

new alerts:

• 1 for Unused import

fixed alerts:

• 3 for Unused import
• 1 for Except block handles 'BaseException'

[blisc]

Just import orders

[lgtm-com]

This pull request fixes 4 alerts when merging 37d99b2 into 7779728 - view on LGTM.com

fixed alerts:

• 3 for Unused import
• 1 for Except block handles 'BaseException'

[lgtm-com]

This pull request fixes 4 alerts when merging 410dbba into 7779728 - view on LGTM.com

fixed alerts:

• 3 for Unused import
• 1 for Except block handles 'BaseException'

[lgtm-com]

This pull request fixes 4 alerts when merging 5c9b906 into 30db6f2 - view on LGTM.com

fixed alerts:

• 3 for Unused import
• 1 for Except block handles 'BaseException'