Package setup refactoring

[DEKHTIARJonathan]

Important Package Refactoring of the collection additional repositories

Project Update:

• Added nemo/package_info.py contains all the core information of the package, including version
• Added Changelog.md, please start using it ;)

To install:

• pip install nemo_toolkit # Core Only
• pip install nemo_toolkit[nlp] # Core + NLP
• pip install nemo_toolkit[nlp,asr] # Core + NLP + ASR
• pip install nemo_toolkit[all] # Absolutely everything

To be noted:

When using pip install -e, you have to use quotes for dev purposes: pip install -e ".[nlp]"

[okuchaiev]

could you please sign your commits https://github.com/NVIDIA/NeMo/pull/289/checks?check_run_id=406125737

[okuchaiev]

Can you please sign your commits so that DCO passes.
Also, please autoformat it so that PEP8 passes, otherwise our CI can't run: http://10.110.42.104:8080/blue/organizations/jenkins/NeMo/detail/PR-289/2/pipeline/22

[lgtm-com]

This pull request introduces 16 alerts and fixes 23 when merging 39a9c0d into b986859 - view on LGTM.com

new alerts:

• 15 for 'import *' may pollute namespace
• 1 for Unused import

fixed alerts:

• 15 for 'import *' may pollute namespace
• 7 for Module is imported with 'import' and 'import from'
• 1 for Unused import

[DEKHTIARJonathan]

OIC yeah I need a GPG key ...

[okuchaiev]

no, you don't need a GPG key. To sign them when you are commiting do "git commit -s" this is different from "git commit -S". DCO's bot output tells you how to sign commited commits.

[blisc]

Can you move README.md back into nemo/ since we already have the repo README as part of README.rst

[blisc]

What is requirements/requirements_test.txt?

[blisc]

Q: nemo_tts requires nemo_asr. Inside requirements_tts.txt, there is a line that says nemo_asr, I am assuming that this no longer works, how do we work around this?

[lgtm-com]

This pull request introduces 16 alerts and fixes 23 when merging b597f23 into b986859 - view on LGTM.com

new alerts:

• 15 for 'import *' may pollute namespace
• 1 for Unused import

fixed alerts:

• 15 for 'import *' may pollute namespace
• 7 for Module is imported with 'import' and 'import from'
• 1 for Unused import

[DEKHTIARJonathan]

Q: nemo_tts requires nemo_asr. Inside requirements_tts.txt, there is a line that says nemo_asr, I am assuming that this no longer works, how do we work around this?

I'll modified this, thanks I did't see :)

[lgtm-com]

This pull request introduces 16 alerts and fixes 23 when merging f2af2e0 into b986859 - view on LGTM.com

new alerts:

• 15 for 'import *' may pollute namespace
• 1 for Unused import

fixed alerts:

• 15 for 'import *' may pollute namespace
• 7 for Module is imported with 'import' and 'import from'
• 1 for Unused import

[lgtm-com]

This pull request introduces 16 alerts and fixes 23 when merging b22ed44 into b986859 - view on LGTM.com

new alerts:

• 15 for 'import *' may pollute namespace
• 1 for Unused import

fixed alerts:

• 15 for 'import *' may pollute namespace
• 7 for Module is imported with 'import' and 'import from'
• 1 for Unused import

[lgtm-com]

This pull request introduces 16 alerts and fixes 23 when merging c84db65 into b986859 - view on LGTM.com

new alerts:

• 15 for 'import *' may pollute namespace
• 1 for Unused import

fixed alerts:

• 15 for 'import *' may pollute namespace
• 7 for Module is imported with 'import' and 'import from'
• 1 for Unused import

[lgtm-com]

This pull request introduces 16 alerts and fixes 23 when merging a2bb05e into b986859 - view on LGTM.com

new alerts:

• 15 for 'import *' may pollute namespace
• 1 for Unused import

fixed alerts:

• 15 for 'import *' may pollute namespace
• 7 for Module is imported with 'import' and 'import from'
• 1 for Unused import

[lgtm-com]

This pull request introduces 16 alerts and fixes 23 when merging e8eca31 into b986859 - view on LGTM.com

new alerts:

• 15 for 'import *' may pollute namespace
• 1 for Unused import

fixed alerts:

• 15 for 'import *' may pollute namespace
• 7 for Module is imported with 'import' and 'import from'
• 1 for Unused import

[lgtm-com]

This pull request introduces 16 alerts and fixes 23 when merging 192cb30 into b986859 - view on LGTM.com

new alerts:

• 15 for 'import *' may pollute namespace
• 1 for Unused import

fixed alerts:

• 15 for 'import *' may pollute namespace
• 7 for Module is imported with 'import' and 'import from'
• 1 for Unused import

[lgtm-com]

This pull request introduces 23 alerts and fixes 28 when merging e424323 into 52bb24d - view on LGTM.com

new alerts:

• 15 for 'import *' may pollute namespace
• 7 for Syntax error
• 1 for Unused import

fixed alerts:

• 15 for 'import *' may pollute namespace
• 7 for Module is imported with 'import' and 'import from'
• 6 for Unused import

[lgtm-com]

This pull request introduces 17 alerts and fixes 17 when merging 2400e8d into 2c50777 - view on LGTM.com

new alerts:

• 15 for 'import *' may pollute namespace
• 1 for Unused import
• 1 for Syntax error

fixed alerts:

• 15 for 'import *' may pollute namespace
• 1 for Unused import
• 1 for Module is imported with 'import' and 'import from'

[lgtm-com]

This pull request introduces 18 alerts and fixes 19 when merging 946c159 into 2c50777 - view on LGTM.com

new alerts:

• 15 for 'import *' may pollute namespace
• 2 for Unused import
• 1 for Module is imported with 'import' and 'import from'

fixed alerts:

• 15 for 'import *' may pollute namespace
• 3 for Unused import
• 1 for Module is imported with 'import' and 'import from'

[lgtm-com]

This pull request introduces 18 alerts and fixes 19 when merging 54a933a into 2c50777 - view on LGTM.com

new alerts:

• 15 for 'import *' may pollute namespace
• 2 for Unused import
• 1 for Module is imported with 'import' and 'import from'

fixed alerts:

• 15 for 'import *' may pollute namespace
• 3 for Unused import
• 1 for Module is imported with 'import' and 'import from'