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

Modified notebooks #51

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Modified notebooks #51

wants to merge 8 commits into from

Conversation

BrasD99
Copy link

@BrasD99 BrasD99 commented Jul 22, 2023

  1. Added a cell with installing dependencies (I saw several issues about this);
  2. Saving .npz goes to the current directory. I also saw an issue with this. To avoid confusion, we pass the full path;
  3. The author of bark-voice-cloning-HuBERT-quantizer changed the name of the folder from hubert to bark_hubert_quantizer;
  4. Running the line tokenizer = CustomTokenizer.load_from_checkpoint('data/models/hubert/tokenizer.pth').to(device) on cpu (e.g. in colab) will result in an error: RuntimeError: Attempting to deserialize object on a CUDA device but torch.cuda.is_available() is False. If you are running on a CPU-only machine, please use torch.load with map_location=torch.device('cpu') to map your storages to the CPU. Added map_location forwarding to the method.

@dagshub
Copy link

dagshub bot commented Jul 22, 2023

@BrasD99 BrasD99 changed the title Modified notebook Modified notebooks Jul 22, 2023
@SethG911
Copy link

You have an error, you need to define "output_path"

@BrasD99
Copy link
Author

BrasD99 commented Jul 31, 2023

@SethG911 I fixed this bug

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.

2 participants