-
Notifications
You must be signed in to change notification settings - Fork 8
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
I think it is wrong code, please confirm this #8
Comments
Hello, Dear author You can import the Dataset module using “from dataset import Dataset” in train.py, it’s correct cause you don‘t need to run the dataset.py file anymore. Thus, there is no required step of running the data.py file in the preparation alone. you can ignore the open ("./config/LJSpeech/preprocess.yaml", "r") cause we did not use this dataset. ("./config/LJSpeech/preprocess.yaml", "r") Source: |
Thank you for your kind response.
2.Is raw_path where the original mp4 data(chem youtube video) should be?
|
Hello, sorry for the delayed response. Q1 Q2.1 Q2.2 No, you do not need to paste. Because preprocessed_path is only related to audio processing, "HPDubbing-how-to-get-face-and-lip" is related to video preprocessing. In "HPDubbing-how-to-get-face-and-lip", we provide some examples and codes of how to extract lip areas and facial areas. The processing flow needs some time, so we directly provide features and disclose the extracted mouth and facial areas (.jpg) of the two datasets chem and V2C. Thanks for your reply again, I will reply to you as soon as possible if I have time. |
first, I did run dataset.py for preparing dataset.
and than,,
um, I found that line number 377 in dataset.py is wrong.
because, in your code...
there is not utils/utils.
so I think that from utils.tools import to_device.
is this correct?
also, I can't find this location
open("./config/LJSpeech/preprocess.yaml", "r")
where is it?
and,, what is it?
The text was updated successfully, but these errors were encountered: