Skip to content

Latest commit

 

History

History
259 lines (190 loc) · 14.5 KB

CONTRIBUTING.md

File metadata and controls

259 lines (190 loc) · 14.5 KB
Raw

How to contribute to ESPnet

1. What to contribute

If you are interested in contributing to ESPnet, your contributions will fall into three categories: major features, minor updates, and recipes.

1.1 Major features

If you want to ask or propose a new feature, please first open a new issue with the tag Feature request or directly contact Shinji Watanabe shinjiw@ieee.org or other main developers. Each feature implementation and design should be discussed and modified according to ongoing and future works. You can find ongoing major development plans at https://github.com/espnet/espnet/milestones or in https://github.com/espnet/espnet/issues (pinned issues)

1.2 Minor Updates (minor feature, bug-fix for an issue)

If you want to propose a minor feature, update an existing minor feature, or fix a bug, please first take a look at the existing issues and/or pull requests. Pick an issue and comment on the task that you want to work on this feature.

If you need help or additional information to propose the feature, you can open a new issue with the tag Discussion and ask ESPnet members.

1.3 Recipes

ESPnet provides and maintains many example scripts, called recipes, that demonstrate how to use the toolkit. The recipes for ESPnet1 are put under egs directory, while ESPnet2 ones are put under egs2. Similar to Kaldi, each subdirectory of egs and egs2 corresponds to a corpus that we have example scripts for.

1.3.1 ESPnet1 recipes

ESPnet1 recipes (egs/X) follow the convention from Kaldi and may rely on several utilities available in Kaldi. As such, porting a new recipe from Kaldi to ESPnet is natural, and the user may refer to port-kaldi-recipe and other existing recipes for new additions. For the Kaldi-style recipe architecture, please refer to Prepare-Kaldi-Style-Directory.

For each recipe, we ask you to report the following: experiments results and environnement, model information. For reproducibility, a link to upload the pre-trained model may also be added. All this information should be written in a markdown file called RESULTS.md and put at the recipe root. You can refer to tedlium2-example for an example.

To generate RESULTS.md for a recipe, please follow the following instructions:

  • Execute ~/espnet/utils/show_result.sh at the recipe root (where run.sh is located). You'll get your environment information and evaluation results for each experiment in a markdown format. From here, you can copy or redirect text output to RESULTS.md.
  • Execute ~/espnet/utils/pack_model.sh at the recipe root to generate a packed ESPnet model called model.tar.gz and output model information. Executing the utility script without argument will give you the expected arguments.
  • Put the model information in RESULTS.md and model link if you're using a private web storage
  • If you don't have private web storage, please contact Shinji Watanabe shinjiw@ieee.org to give you access to ESPnet storage.

1.3.2 ESPnet2 recipes

ESPnet2's recipes correspond to egs2. ESPnet2 applies a new paradigm without dependencies of Kaldi's binaries, which makes it lighter and more generalized. For ESPnet2, we do not recommend preparing the recipe's stages for each corpus but using the common pipelines we provided in asr.sh, tts.sh, and enh.sh. For details of creating ESPnet2 recipes, please refer to egs2-readme.

The common pipeline of ESPnet2 recipes will take care of the RESULTS.md generation, model packing, and uploading. ESPnet2 models are maintained at Hugging Face and Zenodo (Deprecated). You can also refer to the document in https://github.com/espnet/espnet_model_zoo To upload your model, you need first (This is currently deprecated , uploading to Huggingface Hub is prefered) :

  1. Sign up to Zenodo: https://zenodo.org/
  2. Create access token: https://zenodo.org/account/settings/applications/tokens/new/
  3. Set your environment: % export ACCESS_TOKEN=""

To port models from zenodo using Hugging Face hub,

  1. Create a Hugging Face account - https://huggingface.co/
  2. Request to be added to espnet organisation - https://huggingface.co/espnet
  3. Go to egs2/RECIPE/* and run ./scripts/utils/upload_models_to_hub.sh "ZENODO_MODEL_NAME"

To upload models using Huggingface-cli follow the following steps: You can also refer to https://huggingface.co/docs/transformers/model_sharing

  1. Create a Hugging Face account - https://huggingface.co/
  2. Request to be added to espnet organisation - https://huggingface.co/espnet
  3. Run huggingface-cli login (You can get the token request at this step under setting > Access Tokens > espnet token
  4. huggingface-cli repo create your-model-name --organization espnet
  5. git clone https://huggingface.co/username/your-model-name (clone this outside ESPNet to avoid issues as this a git repo)
  6. cd your-model-name
  7. git lfs install
  8. copy contents from exp diretory of your recipe into this directory (Check other models of similar task under ESPNet to confirm your directory structure)
  9. git add .
  10. git commit -m "Add model files"
  11. git push
  12. Check if the inference demo on HF is running successfully to verify the upload

1.3.3 Additional requirements for new recipe

1.3.4 Checklist before you submit the recipe-based PR

  • be careful about the name for the recipe. It is recommended to follow naming conventions of the other recipes
  • common/shared files are linked with soft link (see Section 1.3.3)
  • modified or new python scripts should be passed through latest black formatting (by using python package black). The command to be executed could be black espnet espnet2 test utils setup.py egs*/*/*/local egs2/TEMPLATE/*/pyscripts tools/*.py ci/*.py
  • modified or new python scripts should be passed through latest isort formatting (by using python package isort). The command to be executed could be isort espnet espnet2 test utils setup.py egs*/*/*/local egs2/TEMPLATE/*/pyscripts tools/*.py ci/*.py
  • cluster settings should be set as default (e.g., cmd.sh conf/slurm.conf conf/queue.conf conf/pbs.conf)
  • update egs/README.md or egs2/README.md with corresponding recipes
  • add corresponding entry in egs2/TEMPLATE/db.sh for a new corpus
  • try to simplify the model configurations. We recommend to have only the best configuration for the start of a recipe. Please also follow the default rule defined in Section 1.3.3
  • large meta-information for a corpus should be maintained elsewhere other than in the recipe itself
  • recommend to also include results and pre-trained model with the recipe

2 Pull Request

If your proposed feature or bugfix is ready, please open a Pull Request (PR) at https://github.com/espnet/espnet or use the Pull Request button in your forked repo. If you're not familiar with the process, please refer to the following guides:

3 Version policy and development branches

We basically develop in the master branch.

  1. We will keep the first version digit 0 until we have some super major changes in the project organization level.

  2. The second version digit will be updated when we have major updates, including new functions and refactoring, and their related bug fix and recipe changes. This version update will be done roughly every half year so far (but it depends on the development plan).

  3. The third version digit will be updated when we fix serious bugs or accumulate some minor changes, including recipe related changes periodically (every two months or so).

4 Unit testing

ESPnet's testing is located under test/. You can install additional packages for testing as follows:

$ cd <espnet_root>
$ . ./tools/activate_python.sh
$ pip install -e ".[test]"

4.1 Python

Then you can run the entire test suite using flake8, autopep8, black, isort and pytest with coverage by

./ci/test_python.sh

Followings are some useful tips when you are using pytest:

  • New test file should be put under test/ directory and named test_xxx.py. Each method in the test file should have the format def test_yyy(...). Pytest will automatically find and test them.
  • We recommend adding several small test files instead of grouping them in one big file (e.g.: test_e2e_xxx.py). Technically, a test file should only cover methods from one file (e.g.: test_transformer_utils.py to test transformer_utils.py).
  • To monitor test coverage and avoid the overlapping test, we recommend using pytest --cov-report term-missing <test_file|dir> to highlight covered and missed lines. For more details, please refer to coverage-test.
  • We limited test running time to 2.0 seconds (see: pytest-timeouts). As such, we recommend using small model parameters and avoiding dynamic imports, file access, and unnecessary loops. If a unit test needs more running time, you can annotate your test with @pytest.mark.execution_timeout(sec).
  • For test initialization (parameters, modules, etc), you can use pytest fixtures. Refer to pytest fixtures for more information.

In addition, please follow the PEP 8 convention for the coding style and Google's convention for docstrings. Below are some specific points that should be taken care of in particular:

  • import ordering
  • Avoid writing python2-style code. For example, super().__init__() is preferred over super(CLASS_NAME, self).__init()__.

4.2 Bash scripts

You can also test the scripts in utils with bats-core and shellcheck.

To test:

./ci/test_shell.sh

5 Integration testing

Write new integration tests in ci/test_integration_espnet1.sh or ci/test_integration_espnet2.sh when you add new features in espnet/bin or espnet2/bin, respectively. They use our smallest dataset egs/mini_an4 or egs2/mini_an4 to test run.sh. Don't call python directly in integration tests. Instead, use coverage run --append as a python interpreter. Especially, run.sh should support --python ${python} to call the custom interpreter.

# ci/test_integration_espnet{1,2}.sh

python="coverage run --append"

cd egs/mini_an4/your_task
./run.sh --python "${python}"

5.1 Configuration files

6 Writing new tools

You can place your new tools under

  • espnet/bin: heavy and large (e.g., neural network related) core tools.
  • utils: lightweight self-contained python/bash scripts.

For utils scripts, do not forget to add help messages and test scripts under test_utils.

6.1 Python tools guideline

To generate doc, do not forget def get_parser(): -> ArgumentParser in the main file.

#!/usr/bin/env python3
# Copyright XXX
#  Apache 2.0  (http://www.apache.org/licenses/LICENSE-2.0)
import argparse

# NOTE: do not forget this
def get_parser():
    parser = argparse.ArgumentParser(
        description="awsome tool",  # DO NOT forget this
    )
    ...
    return parser

if __name__ == '__main__':
    args = get_parser().parse_args()
    ...

6.2 Bash tools guideline

To generate doc, support --help to show its usage. If you use Kaldi's utils/parse_option.sh, define help_message="Usage: $0 ...".

7 Writing documentation

See doc.

8 Adding pretrained models

Pack your trained models using utils/pack_model.sh and upload it here (You require permission). Add the shared link to utils/recog_wav.sh or utils/synth_wav.sh as follows:

    "tedlium.demo") share_url="https://drive.google.com/open?id=1UqIY6WJMZ4sxNxSugUqp3mrGb3j6h7xe" ;;

The model name is arbitrary for now.

9 On CI failure

9.1 Travis CI and Github Actions

  1. read the log from PR checks > details

9.2 Circle CI

  1. read the log from PR checks > details
  2. turn on Rerun workflow > Rerun job with SSH
  3. open your local terminal and ssh -p xxx xxx (check circle ci log for the exact address)
  4. try anything you can to pass the CI

9.3 Codecov

  1. write more tests to increase coverage
  2. explain to reviewers why you can't increase coverage