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

Add Flower Baseline: Wav2Vec2.0 TED-LIUM 3 #1780

Closed
18 of 20 tasks
charlesbvll opened this issue Apr 12, 2023 · 1 comment
Closed
18 of 20 tasks

Add Flower Baseline: Wav2Vec2.0 TED-LIUM 3 #1780

charlesbvll opened this issue Apr 12, 2023 · 1 comment
Labels

Comments

@charlesbvll
Copy link
Member

charlesbvll commented Apr 12, 2023

Paper

Nguyen et al., 2023, Federated Learning for ASR based on Wav2vec 2.0, TED-LIUM 3

Link

https://arxiv.org/abs/2302.10790

Maybe give motivations about why the paper should be implemented as a baseline.

No response

Is there something else you want to add?

No response

Implementation

To implement this baseline, it is recommended to do the following items in that order:

For first time contributors

Prepare - understand the scope

  • Read the paper linked above
  • Create the directory structure in Flower Baselines (just the __init__.py files and a README.md)
  • Before starting to write code, write down all of the specs of this experiment in a README (dataset, partitioning, model, number of clients, all hyperparameters, …)
  • Open a draft PR

Implement - make it work

  • Implement some form of dataset loading and partitioning in a separate dataset.py (doesn’t have to match the paper exactly)
  • Implement the model in PyTorch
  • Write a test that shows that the model has the number of parameters mentioned in the paper
  • Implement the federated learning setup outlined in the paper, maybe starting with fewer clients
  • Plot accuracy and loss
  • Run it and check if the model starts to converge

Align - make it converge

  • Implement the exact data partitioning outlined in the paper
  • Use the exact hyperparameters outlined in the paper
  • Make it converge to roughly the same accuracy that the paper states
  • Commit the final hyperparameters and plots
  • Mark the PR as ready
@danieljanes
Copy link
Member

Resolved in #2551

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants