Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactoring howl/data dir #85

Merged
merged 16 commits into from
Oct 24, 2021
Merged

Refactoring howl/data dir #85

merged 16 commits into from
Oct 24, 2021

Conversation

ljj7975
Copy link
Member

@ljj7975 ljj7975 commented May 1, 2021

#65
reorganize the files under howl/data
I have split some files and grouped some relevant files
formatting handled by pre-commit

There was no code change other than some renaming

Since this change is made upon #83, and the change is quite huge
so please review #83 and this one individually

The files are organized as follow

├── common
│   ├── batch.py
│   ├── example.py
│   ├── frame.py
│   ├── __init__.py
│   ├── labeler.py
│   ├── metadata.py
│   ├── phone.py
│   ├── searcher.py
│   ├── tokenizer.py
│   └── vocab.py
├── dataloader.py
├── dataset
│   ├── common_voice_dataset_loader.py
│   ├── dataset_loader.py
│   ├── dataset.py
│   ├── dataset_writer.py
│   ├── gsc_dataset_loader.py
│   ├── hey_snips_dataset_loader.py
│   └── __init__.py
├── __init__.py
├── stitcher.py
└── transform
    ├── batchifier.py
    ├── __init__.py
    ├── meyda.py
    ├── operator.py
    └── transform.py

@ljj7975 ljj7975 requested review from daemon and edwinzhng May 1, 2021 01:26
Base automatically changed from phone to master May 4, 2021 01:08
@ljj7975 ljj7975 merged commit 39d2146 into master Oct 24, 2021
@ljj7975 ljj7975 deleted the refactoring branch October 24, 2021 23:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant